A user received an email due to a commit being pushed upstream and was confused why they got an email. They were not involved in an associated code review. After inspecting with /mail it was due to a Herald rule they configured a year or so back and had forgotten about. The Herald rule observes commits in a repository with an action to email (not subscribe).
Currently the contents of the email don't indicate why they received the email if it's the result of a Herald rule sending an email. Other similar actions do have an indication ("Herald added an auditor: Cat Watch"), though they do not specify which rule. Having something similar for email actions would avoid confusion (ex "Herald rule H20 emailed: you").
Steps to Reproduce
- Create Personal Herald Rule on Commits
- When all of "Diff content" contains "some obscure word only said every 4 blue moons"
- Send me an email
- Create commit with obscure word and push to repository
- Review email received, note that the rule maker is listed as CC
coder committed rAd6d3ad6f80d90d: Task #1 - Nothing to see here. Task #1 - Nothing to see here. Summary: Nothing changed Test Plan: Nobody tests anymore Reviewers: reviewer Reviewed By: reviewer Differential Revision: https://phabricator.company.com/D1 BRANCHES default AFFECTED FILES /project/src/dubious/nolook/TypesOfMoons.java USERS coder (Author) COMMIT https://phabricator.company.com/rAd6d3ad6f80d90d EMAIL PREFERENCES https://repo.mimsoftware.com/settings/panel/emailpreferences/ To: coder Cc: rulemaker
Environment
libphutil | 24ede7a5dbfd38079c87fc61de64012551965837 |
arcanist | 822bc53ca306e06314560d8a76f68771d732e8e0 |
phabricator | 56dd1b297c3e5cdbb477acc7435d6aa5749f33f2 |