Closed Bug 1749747 Opened 2 years ago Closed 2 years ago

Add a commit message hook that prevents landing WIP commits

Categories

(Developer Services :: Mercurial: hg.mozilla.org, task)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: evilpie, Assigned: evilpie)

References

Details

Attachments

(1 file)

I have been noticed a few recent landings starting with "WIP:".

Assignee: nobody → evilpies
Status: NEW → ASSIGNED
Attachment #9258682 - Attachment description: WIP: Bug 1749747 - Don't allow commit messages starting with WIP: → Bug 1749747 - Don't allow commit messages starting with WIP. r?sheehan
See Also: → 1749754

Pushed by cosheehan@mozilla.com:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/354aaa43211a
Don't allow commit messages starting with WIP. r=sheehan

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Pushed by cosheehan@mozilla.com:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/afba7cb37729
hghooks: add tests for wip commit message checker
Flags: needinfo?(sheehan)

I have to deploy it, which I am about to do now. :)

Sorry for the holdup.

Flags: needinfo?(sheehan)

Oh, don't worry. I had wrongly assumed that after landing the commit it would automatically apply.

Should be deployed now, let me know if you see any other issues pop up.

I added a check to Lando as well so users should see a warning about the erroneous WIP before they try and land, and if they do so anyways, your hook check will block the push. :)

Edit: once the Lando patch is deployed to prod*

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: