Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

Add slack notifications for Travis builds #122

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

rhoegg
Copy link

@rhoegg rhoegg commented Feb 28, 2017

No description provided.

Copy link

@smurphy8 smurphy8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there some where this was tested or do we need to merge to test?

@rhoegg
Copy link
Author

rhoegg commented Mar 2, 2017

Yes, sort of? :) Turns out the encrypted token is tied to the repository path, so I couldn't test the exact PR in my own repo. I tested with my own token on a separate branch "ryan-test-build"

https://travis-ci.org/rhoegg/openbudgetokc/branches

@smurphy8
Copy link

I think this is a cool idea, probably needs to be done at a hack night virtual or otherwise so that we can make sure it is all working.

@rhoegg
Copy link
Author

rhoegg commented Mar 20, 2017 via email

@smurphy8
Copy link

smurphy8 commented Mar 20, 2017 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants