-
Notifications
You must be signed in to change notification settings - Fork 8
[RFC] Unify handler-slack.rb and handler-slack-multichannel.rb? #128
Description
On my PR for handler-slack-multichannel.rb (see here) I modified the channel composition from the original created by Rob Wilson in his initial commit (99e1baa9) so that the handler accepts both arrays of channels and strings (single channel), which is the standard in the handler-slack.rb.
While discussing about the PR I realized that if we port this support for both strings and arrays to the handler-slack.rb it would be no longer necessary to keep 2 versions of the handler, which have already diverged in some ways (notable example: handling of non-standard status codes in the message composition, which was breaking the execution), making it difficult to maintain both properly.
It also makes it hard to justify to users to have both versions set in their system asking them to use one or the other depending on the scenario.
I think it would be easier to just port this (should be easy) and completely remove handler-slack-multichannel.rb.