Page MenuHomePhabricator

Use more modern detection of repositories in "arc close-revision"
ClosedPublic

Authored by epriestley on Mar 14 2014, 12:38 AM.
Tags
None
Referenced Files
F13203402: D8523.diff
Tue, May 14, 11:42 PM
F13199659: D8523.id20226.diff
Mon, May 13, 4:49 PM
Unknown Object (File)
Thu, Apr 25, 3:18 AM
Unknown Object (File)
Apr 12 2024, 1:45 PM
Unknown Object (File)
Apr 11 2024, 10:56 AM
Unknown Object (File)
Apr 2 2024, 9:47 PM
Unknown Object (File)
Mar 19 2024, 4:07 PM
Unknown Object (File)
Mar 5 2024, 7:59 AM
Subscribers

Details

Summary

Fixes T4603. We fire arc close-revision --finalize implicitly from arc land, which may close a corresponding Differential revision.

We want to close if the repository is not present in Phabricator (i.e., we'll never be able to close in response to the commit message, since we'll never see it). Historically, we used Arcanist Project -> "Tracked" to make this determination. Instead, just check if the working copy is associated with a repository. This is simpler, easier, and works better.

Test Plan

Ran arc close-revision --finalize.

Diff Detail

Repository
rARC Arcanist
Branch
amend2
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

epriestley retitled this revision from to Use more modern detection of repositories in "arc close-revision".
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 14 2014, 12:42 AM
epriestley updated this revision to Diff 20226.

Closed by commit rARC1e6d958b27df (authored by @epriestley).