Redistribute responsibilities for sending out notifications

betagouv / betaGouvBot

Automated assistant for beta.gouv.fr administrative tasks on slack.

https://beta.gouv.fr

MIT License

4 stars 10 forks source link

Closed Morendil closed 7 years ago

bonjourmauko commented 7 years ago

I really like this PR. See my comments.

bonjourmauko commented 7 years ago

GTM ?

Morendil commented 7 years ago

Other than the nitpick on date_with_default, LGTM.

bonjourmauko commented 7 years ago

rebase + fixup

Morendil commented 7 years ago

Beware katamari PRs. 😉

bonjourmauko commented 7 years ago

I'm going to merge, appart rubocop and the requires, it holds true to description.