Page MenuHomePhabricator

[herald] Add support for Arcanist Project as a field for Differential revisions
ClosedPublic

Authored by neal on Mar 9 2014, 2:28 AM.
Tags
None
Referenced Files
F13208325: D8463.id20135.diff
Thu, May 16, 1:41 PM
F13194196: D8463.diff
Sun, May 12, 9:23 PM
F13181849: D8463.id20135.diff
Thu, May 9, 4:18 PM
F13181847: D8463.id20070.diff
Thu, May 9, 4:18 PM
F13181821: D8463.id.diff
Thu, May 9, 4:05 PM
F13181805: D8463.diff
Thu, May 9, 3:53 PM
F13179035: D8463.diff
Wed, May 8, 8:53 PM
Unknown Object (File)
Sun, May 5, 3:53 AM

Details

Summary

Useful in cases where there is an Arcanist Project but not a repository tracked by Phabricator for a particular revision.

Test Plan

Created a new rule to flag Differential revisions with a particular Arcanist project, verified that it applied as expected via the test console to revisions with the project specified and with a different project specified.

Diff Detail

Repository
rP Phabricator
Branch
add-arcanist-project-to-herald-rules
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

epriestley edited edge metadata.

We're generally trying to move away from Arcanist Projects in as many cases as possible (since for most use cases they're confusing), but the case you describe is one that we can't really get out of. I don't anticipate ever being able to get rid of them either.

This revision is now accepted and ready to land.Mar 11 2014, 8:14 PM
epriestley updated this revision to Diff 20135.

Closed by commit rP8818252f525c (authored by @neal, committed by @epriestley).