Page MenuHomePhabricator
Differential D20957

Add "Author's Packages" and "Committer's Packages" Herald rules for Commits and Hooks
ClosedPublic
Actions

Authored by epriestley on Jan 29 2020, 10:56 PM.

Details

Reviewers
None
Maniphest Tasks
T13480: Implement "Author's packages", "Added project tags", and "Removed project tags" Herald fields
Commits
rPc99485e8a00d: Add "Author's Packages" and "Committer's Packages" Herald rules for Commits and…
Summary

Fixes T13480. Adds the remaining missing Owners package rules for Herald commit adapters.

Test Plan

Created hooks which care about these fields, pushed commits, saw sensible transcript values.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Jan 29 2020, 10:56 PM
Harbormaster completed remote builds in B23751: Diff 49925.Jan 29 2020, 10:58 PM
epriestley requested review of this revision.Jan 29 2020, 10:58 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jan 29 2020, 11:52 PM
Closed by commit rPc99485e8a00d: Add "Author's Packages" and "Committer's Packages" Herald rules for Commits and… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPc99485e8a00d: Add "Author's Packages" and "Committer's Packages" Herald rules for Commits and….

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
herald/
DiffusionCommitAuthorPackagesHeraldField.php
DiffusionCommitAuthorProjectsHeraldField.php
18 lines
6 lines
DiffusionCommitCommitterPackagesHeraldField.php
DiffusionCommitAuthorProjectsHeraldField.php
18 lines
8 lines
DiffusionPreCommitContentAuthorPackagesHeraldField.php
DiffusionPreCommitContentCommitterProjectsHeraldField.php
21 lines
9 lines
DiffusionPreCommitContentCommitterPackagesHeraldField.php
DiffusionPreCommitContentCommitterProjectsHeraldField.php
21 lines
9 lines

Diff 49931

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionCommitAuthorPackagesHeraldField.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionCommitAuthorProjectsHeraldField.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionCommitCommitterPackagesHeraldField.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionCommitCommitterProjectsHeraldField.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionPreCommitContentAuthorPackagesHeraldField.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionPreCommitContentAuthorProjectsHeraldField.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionPreCommitContentCommitterPackagesHeraldField.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionPreCommitContentCommitterProjectsHeraldField.php

Loading...
Phabricator · Built by Phacility