Closed Bug 2016153 Opened 6 months ago Closed 6 months ago

`moz-phab uplift` should strip `DONTBUILD` from commit message

Categories

(Conduit :: moz-phab, defect, P3)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sheehan, Assigned: sheehan)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

See https://phabricator.services.mozilla.com/D282791 as an example.

The original patch landed with the DONTBUILD flag set in the commit message. This was then added to the uplift revision, which would then cause the flag to be added to the landing to the uplift train.

moz-phab uplift should strip DONTBUILD from the commit message before creating the revision in Phabricator.

We could also have Lando remove DONTBUILD from the commit message before landing, but doing this at the moz-phab level means the flag won't appear in the revision on Phabricator and cause confusion.

Assignee: nobody → sheehan

Strip DONTBUILD flag from the commit message title during
moz-phab uplift, as landing on the uplift train may require
a build, unlike the original landing.

Attachment #9547588 - Attachment description: uplift: stip `DONTBUILD` flag from title (Bug 2016153) r?shtrom,zeid → uplift: strip `DONTBUILD` flag from title (Bug 2016153) r?shtrom,zeid
Status: NEW → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Blocks: 2020729
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: