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

Authored by epriestley on Apr 3 2018, 5:50 PM.

Tags

None

Referenced Files

	F19813360: D19292.diff
	Wed, Mar 4, 8:50 AM

	F19514494: D19292.id46159.diff
	Jan 14 2026, 2:23 AM

	F18803654: D19292.id46158.diff
	Oct 18 2025, 2:25 AM

	F18757688: D19292.diff
	Oct 5 2025, 7:00 PM

	F18751479: D19292.id.diff
	Oct 4 2025, 11:18 AM

	F18524578: D19292.diff
	Sep 6 2025, 12:59 PM

	F18379374: D19292.id46158.diff
	Aug 28 2025, 3:52 PM

	F18377257: D19292.id46159.diff
	Aug 28 2025, 12:22 PM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

Path

Size

src/

workflow/

ArcanistPatchWorkflow.php

91 lines

Diff 46159

View Options

src/workflow/ArcanistPatchWorkflow.php

Improve argument parsing for "arc patch --revision Dxxx"ClosedPublicActions