tcSlackBuildNotifier icon indicating copy to clipboard operation
tcSlackBuildNotifier copied to clipboard

Do not send notification if failed build is a personal build

Open gwin003 opened this issue 10 years ago • 10 comments
trafficstars

We use Visual Studio pretested commits to TeamCity fairly often, and it would be awesome if this plugin didn't send the notification to slack if the build was a personal one.

gwin003 avatar Sep 24 '15 18:09 gwin003

Aha! Good point. I've never used them myself. Sounds like a pretty trivial fix if you feel game?

On Fri, 25 Sep 2015 6:20 am Adam Gwin [email protected] wrote:

We use Visual Studio pretested commits to TeamCity fairly often, and it would be awesome if this plugin didn't send the notification to slack if the build was a personal one.

— Reply to this email directly or view it on GitHub https://github.com/PeteGoo/tcSlackBuildNotifier/issues/55.

PeteGoo avatar Sep 24 '15 19:09 PeteGoo

Sure, I'll give it a shot sometime this weekend and see what I can do. Thanks! :)

gwin003 avatar Sep 24 '15 19:09 gwin003

Or better, send the slack notification to the person associated with the personal build.

ghostsquad avatar Nov 09 '15 19:11 ghostsquad

Any progress with this? we are considering to use the teamcity plugin just so we can be able to do this and not have the slack notifier spam us

BHare1985 avatar Dec 21 '15 16:12 BHare1985

surprised to see this didn't already work! would love to have this.

nheath avatar Jul 07 '16 17:07 nheath

will implement this after the refactoring (or maybe just with it)

marvin-w avatar Aug 04 '16 18:08 marvin-w

+1

koerbcm avatar Sep 26 '16 13:09 koerbcm

+1

danieljimenez avatar Nov 08 '16 16:11 danieljimenez

Any update on this issue?

halldorfannar avatar Jan 19 '17 09:01 halldorfannar

If anyone is up for the challenge I think it is as simple as calling isPersonal on SRunningBuild in SlackNotificationListener private void processBuildEvent(SRunningBuild sRunningBuild, BuildStateEnum state)

http://javadoc.jetbrains.net/teamcity/openapi/9.1/jetbrains/buildServer/serverSide/SRunningBuild.html https://github.com/PeteGoo/tcSlackBuildNotifier/blob/master/tcslackbuildnotifier-core/src/main/java/slacknotifications/teamcity/SlackNotificationListener.java

I would do it but I have not touched java for years and I don't have an environment.

bronumski avatar Jul 05 '17 18:07 bronumski