Page MenuHomePhabricator

Don't close revisions on "arc amend"
ClosedPublic

Authored by epriestley on Mar 14 2014, 12:32 AM.
Tags
None
Referenced Files
F13185253: D8522.diff
Sat, May 11, 2:43 AM
Unknown Object (File)
Tue, May 7, 5:03 AM
Unknown Object (File)
Fri, May 3, 2:19 AM
Unknown Object (File)
Wed, Apr 17, 2:40 PM
Unknown Object (File)
Tue, Apr 16, 5:02 PM
Unknown Object (File)
Fri, Apr 12, 1:45 PM
Unknown Object (File)
Apr 11 2024, 7:09 AM
Unknown Object (File)
Apr 11 2024, 7:09 AM
Subscribers

Details

Summary

Ref T4603. This workflow predates arc land and doesn't make much sense in modern Phabricator/Arcanist. It is surprising that arc amend will sometimes close accepted revisions, and we're better at detecting that repositories are tracked, and tracking repositories is easier.

Also fix some inaccuracies and old claims in the documenation and help.

Test Plan

Ran arc amend.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Don't close revisions on "arc amend".
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:41 AM
epriestley updated this revision to Diff 20225.

Closed by commit rARC03ddc1703245 (authored by @epriestley).