Page MenuHomePhabricator
Differential D7820

Add a "branches" rule for Herald commit rules
ClosedPublic
Actions

Authored by epriestley on Dec 23 2013, 7:16 PM.
Tags
None
Referenced Files
F15524091: D7820.diff
Mon, Apr 21, 5:55 AM
F15519202: D7820.id17707.diff
Sat, Apr 19, 8:45 PM
F15515930: D7820.id17696.diff
Fri, Apr 18, 2:38 PM
F15513785: D7820.id.diff
Thu, Apr 17, 9:51 PM
F15510023: D7820.diff
Wed, Apr 16, 6:07 PM
F15430185: D7820.id.diff
Mar 24 2025, 6:23 AM
Unknown Object (File)
Feb 27 2025, 4:20 AM
Unknown Object (File)
Feb 25 2025, 8:51 AM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4195: Implement pre-commit Herald rules
Commits
Restricted Diffusion Commit
rPadcc4ee1db1d: Add a "branches" rule for Herald commit rules
Summary

Fixes T4195. Allows you to write a rule against a commit's branches.

This completes outstanding work on T4195.

Test Plan

Pushed to Git and Mercurial repositories and verified branches were selected correctly by examining transcripts.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley added inline comments.Dec 23 2013, 7:17 PM
src/applications/diffusion/engine/DiffusionCommitHookEngine.php
431

This fixes a bug where we could update the wrong ref when a push includes multiple branches.

btrahan accepted this revision.Dec 26 2013, 6:22 PM
epriestley closed this revision.Dec 26 2013, 6:40 PM

Closed by commit rPadcc4ee1db1d (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
engine/
29 lines
herald/
7 lines
herald/
adapter/
3 lines

Diff 17707

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