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
F14068556: D7820.diff
Tue, Nov 19, 11:01 PM
F14054153: D7820.diff
Fri, Nov 15, 10:52 PM
F14043433: D7820.diff
Tue, Nov 12, 10:11 AM
F14027692: D7820.diff
Fri, Nov 8, 8:51 AM
F13999658: D7820.diff
Thu, Oct 24, 4:15 PM
F13966248: D7820.diff
Oct 16 2024, 7:11 AM
F13966242: D7820.id.diff
Oct 16 2024, 7:08 AM
F13966239: D7820.id17696.diff
Oct 16 2024, 7:07 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