Periodic file updates should have an approval in the commit message

RESOLVED FIXED

Status

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: sfraser, Assigned: sfraser)

Tracking

unspecified

Firefox Tracking Flags

(firefox-esr60 fixed, firefox61 fixed, firefox62 fixed)

Details

Attachments

(2 attachments)

Add the command-line flag for a= and make sure the text in that field is appropriate.
Assignee: nobody → sfraser
Is the text 'repo-update' sufficient for the approval flag, or should it be something else?
Flags: needinfo?(ryanvm)
I've been using a=release so far, but repo-update also sounds fine.
Flags: needinfo?(ryanvm)
I'll update the old review expiry, too, while I'm at it.
Comment on attachment 8986137 [details]
Bug 1468770 repo-update submitter and commit msg r=ryanvm

Ryan VanderMeulen [:RyanVM] has approved the revision.

https://phabricator.services.mozilla.com/D1700
Attachment #8986137 - Flags: review+

Comment 6

Last year
Pushed by sfraser@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/faec64e8d578
repo-update submitter and commit msg r=RyanVM

Comment 7

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/faec64e8d578
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Looks like this needs "Name <email>" to be truly happy.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment on attachment 8986829 [details]
Bug 1468770 Update author for repo-update commits r=ryanvm

Ryan VanderMeulen [:RyanVM] has approved the revision.

https://phabricator.services.mozilla.com/D1753
Attachment #8986829 - Flags: review+

Comment 13

Last year
Pushed by sfraser@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/47e6dedb5185
Update author for repo-update commits r=RyanVM

Comment 14

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/47e6dedb5185
Status: REOPENED → RESOLVED
Closed: Last yearLast year
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.