Page MenuHomePhabricator
Differential D8502

Add a "tags" field to Diffusion commit
ClosedPublic
Actions

Authored by epriestley on Mar 12 2014, 4:53 PM.
Tags
None
Referenced Files
F18831415: D8502.id20178.diff
Sat, Oct 25, 12:53 PM
F18822106: D8502.id20170.diff
Thu, Oct 23, 4:18 AM
F18808612: D8502.id20170.diff
Sun, Oct 19, 7:29 AM
F18808090: D8502.id.diff
Sun, Oct 19, 4:48 AM
F18806652: D8502.diff
Sat, Oct 18, 8:52 PM
F18531687: D8502.id.diff
Sep 7 2025, 4:58 AM
F18528309: D8502.diff
Sep 6 2025, 8:26 PM
F18291985: D8502.id20178.diff
Aug 23 2025, 11:45 AM
View All Files
Subscribers
aran
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T4588: Add "branches" / "tags" information in Diffusion commit emails
Commits
Restricted Diffusion Commit
rPa4a4322d7abf: Add a "tags" field to Diffusion commit
Summary
  • Fixes T4588.
  • See D8501.
  • Adds a "Tags" field for Herald commit emails.
  • Fixes a bug in tagsquery when filtering by commit name.
  • Make tagsquery just return nothing instead of fataling against Mercurial/Subversion.
Test Plan

Used bin/repository/reparse.php --herald to exercise this code.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20170.Mar 12 2014, 4:53 PM
epriestley retitled this revision from to Add a "tags" field to Diffusion commit.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T4588: Add "branches" / "tags" information in Diffusion commit emails.
Herald added subscribers: epriestley, aran.Mar 12 2014, 4:53 PM
btrahan accepted this revision.Mar 12 2014, 6:16 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 12 2014, 6:16 PM
epriestley closed this revision.Mar 12 2014, 6:30 PM
epriestley updated this revision to Diff 20178.

Closed by commit rPa4a4322d7abf (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
diffusion/
conduit/
17 lines
repository/
customfield/
37 lines
worker/
11 lines

Diff 20178

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/conduit/ConduitAPI_diffusion_tagsquery_Method.php

Loading...
View Options

src/applications/repository/customfield/PhabricatorCommitTagsField.php

Loading...
View Options

src/applications/repository/worker/PhabricatorRepositoryCommitHeraldWorker.php

Loading...
Phabricator · Built by Phacility