Page MenuHomePhabricator

Fix text lint issues
ClosedPublic

Authored by joshuaspence on Feb 3 2015, 10:20 AM.
Tags
None
Referenced Files
F13224724: D11640.id28292.diff
Sun, May 19, 10:25 AM
F13224647: D11640.id28293.diff
Sun, May 19, 9:49 AM
F13196171: D11640.diff
Sun, May 12, 11:01 PM
Unknown Object (File)
Tue, May 7, 1:45 PM
Unknown Object (File)
Tue, May 7, 12:56 AM
Unknown Object (File)
Sun, May 5, 12:49 PM
Unknown Object (File)
Fri, May 3, 8:29 AM
Unknown Object (File)
Fri, May 3, 3:16 AM
Subscribers

Details

Summary

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

Test Plan

arc lint --everything

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
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.
src/lint/linter/ArcanistTextLinter.php
18 ↗(On Diff #28017)

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

src/lint/linter/ArcanistTextLinter.php
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.