Page MenuHomePhabricator

Improve handling of options for `arc` workflows
ClosedPublic

Authored by joshuaspence on Jan 22 2015, 12:28 AM.
Tags
None
Referenced Files
F19802882: D11464.id.diff
Sun, Mar 1, 7:28 PM
F19514117: D11464.diff
Jan 13 2026, 10:20 PM
F19513744: D11464.id27560.diff
Jan 13 2026, 4:51 PM
F19513700: D11464.id.diff
Jan 13 2026, 4:15 PM
F19512599: D11464.id27560.diff
Jan 12 2026, 11:17 PM
F19490999: D11464.diff
Jan 8 2026, 7:10 AM
F19324824: D11464.id27560.diff
Dec 25 2025, 1:44 PM
F19323630: D11464.diff
Dec 25 2025, 12:26 PM
Subscribers

Details

Summary

Allow --severity=warning to mean the same as --severity warning. Longer term, we should convert this code to use PhutilArgumentParser, although it doesn't seem that PhutilArgumentParser support British spelling ;)

Test Plan

Ran arc lint --severity=warning. Also var_dumped $args to make sure it looked reasonable.

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3983
Build 3996: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Improve handling of options for `arc` workflows.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 22 2015, 8:04 PM
This revision was automatically updated to reflect the committed changes.