Page MenuHomePhabricator
Differential D7809

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

Authored by epriestley on Dec 20 2013, 12:23 AM.
Tags
None
Referenced Files
F13162895: D7809.diff
Mon, May 6, 4:18 PM
Unknown Object (File)
Sat, May 4, 12:18 PM
Unknown Object (File)
Fri, May 3, 3:30 AM
Unknown Object (File)
Mon, Apr 29, 2:44 PM
Unknown Object (File)
Wed, Apr 24, 10:14 PM
Unknown Object (File)
Sat, Apr 20, 6:12 PM
Unknown Object (File)
Wed, Apr 10, 4:21 AM
Unknown Object (File)
Apr 4 2024, 1:11 AM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4195: Implement pre-commit Herald rules
Commits
Restricted Diffusion Commit
rPa64d127e2569: Add "is merge commit" Herald field for pre-commit rules
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

Branch
hashq6
Lint
Lint Passed
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
engine/
13 lines
herald/
23 lines
herald/
adapter/
17 lines

Diff 17666

View Options

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Loading...
View Options

src/applications/diffusion/herald/HeraldPreCommitContentAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
Phabricator · Built by Phacility