feature: configuration request: list "person who broke the build" on all build failures

PeteGoo / tcSlackBuildNotifier

Posts build notifications to Slack

MIT License

311 stars 81 forks source link

feature: configuration request: list "person who broke the build" on all build failures #34

Open gwynnebaer opened 9 years ago

gwynnebaer commented 9 years ago

Currently the person who broke the build only appears on the first failure.

We would like to see the user listed on all failures (so we don't have to hunt them down if we re-run the build for some reason to check some other issue).

Thank you.

PeteGoo commented 9 years ago

Hi, thanks for the feedback. The reason I chose this option was to stop a sequence of failures spamming everyone who committed subsequent to the problem but that .

I'm currently battling two factors here:

Which is the lesser of two evils above?
How do I stop the configuration page being "option overload"

Thoughts?

gwynnebaer commented 9 years ago

My opinion: checkbox for displaying "commit info" (user, commit log) or not Behavior: if checked, all notices contain this information. Unchecked, nothing. I agree there seems to be a loop issue (all of the notices show up twice in Slack, but wasn't sure yet if that was my fault). And shameless plug for global configuration support (our use of TC is pretty vanilla). :-D

Thanks, -matt