Skip to content

Conversation

ezrast
Copy link

@ezrast ezrast commented Jun 8, 2017

No description provided.

@ezrast
Copy link
Author

ezrast commented Jun 8, 2017

Haha whoops, I meant to PR this to master on my own fork and do some more testing before sending it to you.

But anyway I added the following things to your thing:

  • can specify multiple Slack channels with an array
  • can override settings on a per-branch basis by turning branches_to_handle into a hash
  • can override the default slack channel for specific build states by specifying a key like "channel-"

I don't think it breaks configuration on any existing deployments but like I said I haven't exactly tested it extensively yet.

Merge if you wanna, don't if my changes suck.

Peace

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.

1 participant