Page MenuHomePhabricator

Add "is merge commit" Herald field for pre-commit rules
ClosedPublic

Authored by epriestley on Dec 20 2013, 12:23 AM.
Tags
None
Referenced Files
F18750837: D7809.id.diff
Sat, Oct 4, 9:02 AM
F18744299: D7809.diff
Fri, Oct 3, 2:29 AM
F18434088: D7809.id.diff
Aug 31 2025, 8:51 AM
F18408181: D7809.diff
Aug 30 2025, 2:11 AM
F18087707: D7809.id17674.diff
Aug 6 2025, 5:10 AM
F17869541: D7809.id.diff
Jul 28 2025, 11:33 AM
F17845663: D7809.diff
Jul 27 2025, 3:50 AM
Unknown Object (File)
Jun 16 2025, 6:34 PM
Subscribers

Details

Summary

Ref T4195. This allows you to write rules which disallow merge commits.

Also make the reject message a little more useful.

Test Plan
remote: This push was rejected by Herald push rule H27.
remote: Change: commit/daed0d448404
remote:   Rule: No Merges
remote: Reason: No merge commits allowed. If you must push a merge, include "@force-merge" in the commit message.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped