Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #39: if option is active, the list of commits is shown... #44

Merged
merged 1 commit into from
Feb 4, 2015
Merged

Conversation

peergum
Copy link
Contributor

@peergum peergum commented Feb 4, 2015

... after ANY build (as opposed to only successful builds previously)

@samrocketman
Copy link
Member

This is patching #30. Just adding it as a reference.

@samrocketman samrocketman self-assigned this Feb 4, 2015
@samrocketman samrocketman modified the milestone: slack-1.7 Feb 4, 2015
@samrocketman
Copy link
Member

Can you amend your commit message so that it has a short title about 50 chars long, a blank line, and then a more descriptive message? e.g.

Patching #30: Always show user list in slack

As opposed to only successful builds previously.

It helps when viewing the git tree from the terminal. Just...

git commit --amend
#amend the commit
git push origin master --force

As opposed to just for successful builds previously.
@peergum
Copy link
Contributor Author

peergum commented Feb 4, 2015

Sure. Thanks for the initiative and good work, Sam!

@samrocketman
Copy link
Member

No problem. FYI I don't plan to do much active development. I'll mostly just be maintaining incoming pull requests and releases.

@samrocketman samrocketman merged commit 3ec7d4f into jenkinsci:master Feb 4, 2015
samrocketman added a commit that referenced this pull request Feb 4, 2015
samrocketman added a commit that referenced this pull request Feb 10, 2015
New features:

* Advanced settings
  * Team domain and token can be specified per project; falls back to
    global config if not specified (#19)
  * Post a list of commits with the build notification (title and author
    of commit) (#30, #44, #45)
  * Include JUnit test summary in build notifications (#25)
* Use colors when sending a message using slack (#20, #24).  Also, the
  start notifications use the color of the last build result (#43)
* Support for authenticated proxies (#14)
* Test Connection button (#46, #28)
* Option to disable subsequent notifications on build failures (#46, #15)

Improvements:

* Report 'return to normal' on transition from unstable to success (#21)
* Improved logging.

Bug fixes:

* When changing the global slack settings the slack config in jobs are
  updated as well (#26, #12)
* Fix NullPointerException and output log message if slack is not
  configured (#35, JENKINS-26066)
alexey-pelykh pushed a commit to brainbeanapps/slack-plugin that referenced this pull request Nov 3, 2016
getKeychain returns a global keychain preferred over a local path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants