-
Notifications
You must be signed in to change notification settings - Fork 178
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
.travis.yml: Enable travis-buddy #551
Conversation
Hey @manankalra, |
#551 (comment) kind of comments can be disabled by turning on travis-buddy only if the tests fail. |
@@ -11,6 +11,7 @@ script: | |||
|
|||
notifications: | |||
email: false | |||
webhooks: https://www.travisbuddy.com/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@manankalra Looks good. 👍
I also agree with you that adding "on_success: never" should be there as well. Gitmate automatically approves the PR if all CI passes, so there should ideally be no message in that case. 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
per review
didn't get you |
@manankalra He just meant to add the changes as suggested by me to the PR - No message if all tests pass 😅 |
36a5180
to
54b819f
Compare
@gitmate-bot rebase |
Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently |
Automated rebase with GitMate.io was successful! 🎉 |
ack 54b819f |
@gitmate-bot ff |
Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently |
Automated fastforward with GitMate.io was successful! 🎉 |
Related to coala/meta#114