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
F22712731: D7820.id17696.diff
Sat, Oct 3, 2:47 PM
F22712722: D7820.id17696.diff
Sat, Oct 3, 2:46 PM
F22518222: D7820.diff
Sun, Sep 20, 11:36 AM
Unknown Object (File)
Sep 1 2026, 12:39 PM
Unknown Object (File)
Jun 28 2026, 5:13 AM
Unknown Object (File)
Jun 22 2026, 6:32 PM
Unknown Object (File)
Jun 6 2026, 11:42 PM
Unknown Object (File)
Apr 2 2026, 8:38 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