Follow-up from "Send email notifications for Generic Alerts"
The following discussions from !24414 (merged) should be addressed:
-
@seanarnold started a discussion: (+4 comments) I extracted this method as Rubocop was now complaining about it.
I think it's nicer extracted too
-
@splattael started a discussion: Do we need to add/adjust documentation as well?
🤔 /cc @aqualls
If so, let's do this in a follow-up MR as well
👍 -
@splattael started a disucssion: Oh, I've missed this during the final review
😞 Could we use
let_it_be(:project, reload: true) { ... }
?
Edited by Sean Arnold