MFH: 2019Q2 (any text)
as a signal not to send the MFH request email.
adamw on Jun 19 2019, 12:25 PM.Authored by
I think ultimately what happens on quarterly branch is still being covered by commit mail? This only affects MFH approval email, which seems to be redundant if it's a hat commit and the intention was not to ask an approval in the first place.
Then if the MFH line contains anything else than a branch name, say:
There is no need to send an email because it will already have gotten committed.
Maybe it should echo the fact that no MFH email is being sent. (So need two tests.)
That's what the above patch does.
That's a great idea. Instant feedback would help alleviate the behaving-silently thing.