Page MenuHomePhabricator

Modularize "arc lint" renderers
ClosedPublic

Authored by epriestley on Sep 25 2017, 9:48 PM.
Tags
None
Referenced Files
F15472544: D18645.id44764.diff
Sat, Apr 5, 2:27 PM
F15446725: D18645.id44784.diff
Thu, Mar 27, 7:57 PM
F15437908: D18645.id.diff
Tue, Mar 25, 10:05 PM
F15428407: D18645.id44764.diff
Sun, Mar 23, 8:09 PM
F15424163: D18645.id44764.diff
Sat, Mar 22, 8:38 PM
F15407815: D18645.diff
Tue, Mar 18, 7:46 PM
F15398456: D18645.id44784.diff
Mon, Mar 17, 12:09 AM
F15397085: D18645.diff
Sun, Mar 16, 6:04 PM
Subscribers
None

Details

Summary

Ref T12996. arc lint can render in different formats with --output <format>.

Previously, these were a big switch() statement and a bunch of hard-coded stuff.

Modularize them so that that third parties could add new renderers, and this code can be refactored toward parallelizing the lint step.

This has a small behavioral change: we no longer hide "autofix" messages. I'd like to generally simplify the number of amend/autofix flags here, so this edges us toward that slightly.

Test Plan

Ran arc lint with all the --output flags in states with warnings and no warnings, saw sensible-seeming behavior.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable