search

enlivenhq / teamcity-slack

Configurable TeamCity notifier plugin for Slack.
137 stars 51 forks source link

Feature request: Show who's change broke the build #10

Open sjincho opened 8 years ago

sjincho commented 8 years ago

It would be great if the plugin could show the authors of the changes in the slack message.

luketheobscure commented 8 years ago

+1

cy6erskunk commented 8 years ago

What if there're 100 commits from 50 authors?

irobinson commented 8 years ago

@cy6erskunk Perhaps we could pull out the list of authors & just truncate the string after a maximum number of characters if that's a concern. I would have to imagine that we could come up with a comfortable maximum such that it would be concise and useful in most circumstances but still usable in those edge cases.

zilionis commented 8 years ago

+1 Maybe should show last committer Also would be great to have custom config to show branch name :)

zilionis commented 8 years ago

Btw, i just updated plugin with that new icons. I saw branch show issue was fixed, but in mine report branch name still missing (I changed output to be more readable :) )

:white_check_mark: PROJECT_NAME #68 built successfully Build Status

Project: PROJECT_NAME Build: 68 Status: built successfully

cy6erskunk commented 8 years ago

@zilionis Default branch name is not displayed in the notifications, maybe it is your case? And why do you need custom config to show branch name, what problem is should solve?

zilionis commented 8 years ago

@cy6erskunk if we have notification in slack with all different branches is nice to see branch (we will use multiple of them) and who broke it :). Just want same message in all cases.

DotNetRockStar commented 7 years ago

+1