Bug 1455698 - implement configuration for wihch reasonResolved to ignore #31
Conversation
config.yml
Outdated
| emailBlacklist: !env:json EMAIL_BLACKLIST | ||
| # ignore tasks with reasonResolved matching something in this list | ||
| ignoreTaskReasonResolved: | ||
| - cancelled |
There was a problem hiding this comment.
This is spelled with one l in taskcluster apparently!
There was a problem hiding this comment.
Which is apparently the American way, so I guess we can't blame @jonasfj :)
There was a problem hiding this comment.
Awesome catch BTW, your review skills are legion :)
There was a problem hiding this comment.
I only noticed because I had just reviewed a jonas patch where it was spelled canceled and I thought, "that's a weird spelling". I was going to tell him he was wrong until I checked the docs :p
There was a problem hiding this comment.
Which is weird because we have two americans here who spell it the wrong way apparently.
|
@djmitche I would have bit this option on task.extra.notify instead... |
|
Just in case you misunderstood my comment in the bug.. |
|
I feel like that's as messy as putting it in the routes. |
3 participants
No description provided.