Page MenuHomePhabricator

Fix text lint issues

Authored by joshuaspence on Feb 3 2015, 10:20 AM.
Referenced Files
F13309215: D11640.id28017.diff
Mon, Jun 10, 9:47 AM
F13308365: D11640.id28294.diff
Mon, Jun 10, 3:05 AM
F13307725: D11640.id28291.diff
Sun, Jun 9, 9:22 PM
F13307596: D11640.id28292.diff
Sun, Jun 9, 8:10 PM
F13300831: D11640.diff
Fri, Jun 7, 10:31 AM
F13285782: D11640.id28339.diff
Tue, Jun 4, 2:40 AM
F13269948: D11640.diff
Wed, May 29, 8:19 AM
F13267169: D11640.id28339.diff
Wed, May 29, 2:18 AM



Ref T5105. This is a proof-of-concept for D11458.

Test Plan

arc lint --everything

Diff Detail

rARC Arcanist
Lint Not Applicable
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Fix text lint issues.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
18 ↗(On Diff #28017)

This causes a behavioral change, right? That's maaaaybe OK but likely surprising for users...

18 ↗(On Diff #28017)

Yes it does, but it seemed necessary to make this work. Otherwise, we can't use max_line_length = in an .editorconfig file to mean "no maximum line length". We could maybe tweak this to use some non-null sentinel value I suppose.

joshuaspence edited edge metadata.

Change end of line character

epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 6 2015, 1:39 PM
This revision was automatically updated to reflect the committed changes.