Page MenuHomePhabricator
Differential D14924

Remove calls to getCallsign() in bin/repository scripts
ClosedPublic
Actions

Authored by epriestley on Jan 2 2016, 2:53 AM.
Tags
None
Referenced Files
F13258171: D14924.id.diff
Sun, May 26, 7:15 AM
F13258170: D14924.id36062.diff
Sun, May 26, 7:15 AM
F13258169: D14924.id36060.diff
Sun, May 26, 7:15 AM
F13231780: D14924.id36062.diff
Tue, May 21, 12:46 AM
F13215226: D14924.diff
Fri, May 17, 4:00 PM
Unknown Object (File)
Mon, May 6, 8:24 AM
Unknown Object (File)
Sun, May 5, 5:43 PM
Unknown Object (File)
May 2 2024, 5:50 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T4245: Make repository URIs slightly prettier
Commits
Restricted Diffusion Commit
rPedcc3232aabc: Remove calls to getCallsign() in bin/repository scripts
Summary

Ref T4245. Prepare these scripts for a callsign-free world. This also makes them more flexible and easier to use.

The following are now valid ways to identify a repository for these scripts: ID (3), PHID (PHID-REPO-wxyz), R<ID> (R3), r<CALLSIGN> (rSKYNET), CALLSIGN (SKYNET).

In the future, a human-readable label (skynet) may also become valid.

Test Plan
  • Ran bin/repository reparse --all ... with rX, X, 3, R3.
  • Ran bin/repository reparse --change ... with rXaaa, including short versions.
  • Ran bin/repository update ... with rX, X, 3, R3.
  • Ran bin/repository refs ... with various identifiers.
  • Ran bin/repository pull ... with various identifiers.
  • Ran bin/repository mirror ... with various identifiers.
  • Ran bin/repository mark-imported ... with various identifiers.
  • Ran bin/repository list.
  • Ran bin/repository importing ... with various identifiers and examined output.
  • Ran bin/repository edit ... with various identifiers.
  • Ran bin/repository discover ... with various identifiers.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36060.Jan 2 2016, 2:53 AM
epriestley retitled this revision from to Remove calls to getCallsign() in bin/repository scripts.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T4245: Make repository URIs slightly prettier.
chad accepted this revision.Jan 2 2016, 2:59 AM
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 2 2016, 2:59 AM
Closed by commit rPedcc3232aabc: Remove calls to getCallsign() in bin/repository scripts (authored by epriestley, committed by epriestley). · Explain WhyJan 2 2016, 12:23 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 36062

View Options

src/applications/repository/management/PhabricatorRepositoryManagementDiscoverWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementEditWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementImportingWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementListWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementMarkImportedWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementMirrorWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementPullWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementRefsWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementReparseWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementUpdateWorkflow.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementWorkflow.php

Loading...
View Options

src/applications/repository/phid/PhabricatorRepositoryRepositoryPHIDType.php

Loading...
View Options

src/applications/repository/query/PhabricatorRepositoryQuery.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
Phabricator · Built by Phacility