Page MenuHomePhabricator
Differential D19292

Improve argument parsing for "arc patch --revision Dxxx"
ClosedPublic
Actions

Authored by epriestley on Apr 3 2018, 5:50 PM.
Tags
None
Referenced Files
F15481121: D19292.id.diff
Tue, Apr 8, 6:20 PM
F15476829: D19292.diff
Mon, Apr 7, 9:44 AM
F15471444: D19292.diff
Sat, Apr 5, 5:00 AM
F15448169: D19292.id46158.diff
Fri, Mar 28, 3:21 AM
F15448168: D19292.id.diff
Fri, Mar 28, 3:21 AM
F15446674: D19292.diff
Thu, Mar 27, 7:43 PM
F15410225: D19292.diff
Wed, Mar 19, 6:55 AM
F15404823: D19292.id46159.diff
Tue, Mar 18, 9:15 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13116: Plans: 2018 Week 14 Bonus Content
Commits
rARCe44a2d3ac0bf: Improve argument parsing for "arc patch --revision Dxxx"
Summary

See PHI527. Ref T13116. The --revision flag currently fails if the argument is in the form D123 instead of 123. Normalize monogram arguments.

Test Plan

Ran arc patch --revision Dxxx, arc patch --revision xxx, arc patch --revision xxx --diff yyy, arc patch; got good behavior on the good ones and sensible error messages on the other ones.

Diff Detail

Repository
rARC Arcanist
Branch
patch1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19979
Build 27093: Run Core Tests
Build 27092: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 3 2018, 5:50 PM
Harbormaster completed remote builds in B19979: Diff 46158.Apr 3 2018, 5:51 PM
epriestley requested review of this revision.Apr 3 2018, 5:51 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 3 2018, 5:56 PM
Closed by commit rARCe44a2d3ac0bf: Improve argument parsing for "arc patch --revision Dxxx" (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
workflow/
91 lines

Diff 46158

View Options

src/workflow/ArcanistPatchWorkflow.php

Loading...
Phabricator · Built by Phacility