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

Added a /jenkins-notifier that works with existing jenkins-notification ... #14

Merged
merged 3 commits into from Dec 21, 2012

Conversation

@b2jrock
Copy link
Contributor

b2jrock commented Dec 20, 2012

@miohtama

This comment has been minimized.

Copy link

miohtama commented on 68053f4 Dec 20, 2012

Just a question here: you should probably use unicode string u"" if you get a conversion error? Did you test this with non-ASCII payloads?

@miohtama

This comment has been minimized.

Copy link
Member

miohtama commented Dec 20, 2012

Also another Q: I have started a new Sphinx based documentation. Would you mind to drop jenkins.rst (copy e.g. zabbix.rst) and explain how to use Jenkins integration?

… the event that commit msg has some trickier data.

Add helpfile.
@b2jrock

This comment has been minimized.

Copy link
Owner

b2jrock commented on f6db650 Dec 21, 2012

You were right on the text conversion issue. I added a help file outlining the basic config.

@tenpn

This comment has been minimized.

Copy link
Contributor

tenpn commented Dec 21, 2012

Wow this is amazing. I've actually spent the better part of three hours trying various plugins to make Jenkins talk to the local sevabot server. Would love this to get into the main repro.

@tenpn

This comment has been minimized.

Copy link
Contributor

tenpn commented Dec 21, 2012

...I know this isn't jenkins tech support, but would there be a way with this end-point link-up of only notifying on failed builds, rather than all builds?

@miohtama

This comment has been minimized.

Copy link
Member

miohtama commented Dec 21, 2012

@tenpn Try open a new issue if you need help. We do not currently have other media to discuss besides the issue tracker.

@miohtama miohtama merged commit f6db650 into opensourcehacker:master Dec 21, 2012
@miohtama

This comment has been minimized.

Copy link
Member

miohtama commented Dec 21, 2012

Done!

askeing pushed a commit to askeing/sevabot that referenced this pull request Jul 2, 2015
gong special power initial version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.