Page MenuHomePhabricator

Make lint tests handle paths better and distinguish between "0" and "null" more carefully
ClosedPublic

Authored by epriestley on May 27 2020, 7:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 8:10 AM
Unknown Object (File)
Sun, Nov 17, 7:47 PM
Unknown Object (File)
Sun, Nov 17, 7:56 AM
Unknown Object (File)
Sun, Nov 17, 3:26 AM
Unknown Object (File)
Fri, Nov 15, 4:20 PM
Unknown Object (File)
Fri, Nov 8, 9:21 PM
Unknown Object (File)
Fri, Nov 8, 6:19 AM
Unknown Object (File)
Fri, Nov 1, 5:38 AM
Subscribers
None

Details

Summary

Ref T13543. Currently, the cpplint tests do not function because cpplint is passed a path which does not end in a suffix it recognizes.

Change the tempfile / path code to pass linter path/to/example.c-style linters a path they expect.

Then, correct some older code which was playing it fast-and-loose with "null" vs "0".

Test Plan

Ran arc unit --everything, got a clean bill of health on all the linters I have installed. (This is probably not all tests, since I have only a subset of linters installed locally that we have code for.)

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.May 27 2020, 7:37 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.