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
F19031344: D21291.diff
Tue, Nov 25, 12:13 AM
F19015748: D21291.diff
Sun, Nov 23, 1:19 AM
F19002192: D21291.diff
Fri, Nov 21, 5:12 AM
F18985496: D21291.id50702.diff
Mon, Nov 17, 1:04 PM
F18778485: D21291.id.diff
Oct 11 2025, 9:39 AM
F18761919: D21291.diff
Oct 6 2025, 4:50 PM
F18755210: D21291.id50703.diff
Oct 5 2025, 4:19 AM
F18741498: D21291.id50702.diff
Oct 2 2025, 1:29 PM
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.