Notify is becoming a bit overloaded and its technically wrong in hindsight, you can send someone all sorts of slack messages with that client for purposes beyond a notification.
You'll probably need to tweak the tests a bit to make them work with the change of nomenclature.
Can we also add a repo in the /slack/ directory with instructions on how to use the client. Simple example snippet or two to show usage, the same sort of thing you'd find on python packages on github.
What is this
PR the slack client from:
What to do
More or less copy paste, but can we:
SlackMessengerNotify is becoming a bit overloaded and its technically wrong in hindsight, you can send someone all sorts of slack messages with that client for purposes beyond a notification.
You'll probably need to tweak the tests a bit to make them work with the change of nomenclature.
Can we also add a repo in the
/slack/directory with instructions on how to use the client. Simple example snippet or two to show usage, the same sort of thing you'd find on python packages on github.Acceptance Criteria