Closed
Bug 1468770
Opened 6 years ago
Closed 6 years ago
Periodic file updates should have an approval in the commit message
Categories
(Release Engineering :: Release Automation: Other, enhancement)
Release Engineering
Release Automation: Other
Tracking
(firefox-esr60 fixed, firefox61 fixed, firefox62 fixed)
RESOLVED
FIXED
People
(Reporter: sfraser, Assigned: sfraser)
Details
Attachments
(2 files)
Add the command-line flag for a= and make sure the text in that field is appropriate.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → sfraser
Assignee | ||
Comment 1•6 years ago
|
||
Is the text 'repo-update' sufficient for the approval flag, or should it be something else?
Flags: needinfo?(ryanvm)
Comment 2•6 years ago
|
||
I've been using a=release so far, but repo-update also sounds fine.
Flags: needinfo?(ryanvm)
Assignee | ||
Comment 3•6 years ago
|
||
I'll update the old review expiry, too, while I'm at it.
Comment 4•6 years ago
|
||
Comment 5•6 years ago
|
||
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+
Pushed by sfraser@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/faec64e8d578 repo-update submitter and commit msg r=RyanVM
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/faec64e8d578
Comment 8•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-release/rev/9f2e3a37b12f
status-firefox61:
--- → fixed
Comment 9•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr60/rev/4ad4c31db9b1
status-firefox-esr60:
--- → fixed
Assignee | ||
Comment 10•6 years ago
|
||
Looks like this needs "Name <email>" to be truly happy.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 11•6 years ago
|
||
Comment 12•6 years ago
|
||
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•6 years ago
|
||
Pushed by sfraser@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/47e6dedb5185 Update author for repo-update commits r=RyanVM
Comment 14•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/47e6dedb5185
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
Comment 15•6 years ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-release/rev/59666a63d700 https://hg.mozilla.org/releases/mozilla-esr60/rev/c223777b7899
You need to log in
before you can comment on or make changes to this bug.
Description
•