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
F19745497: D21291.id.diff
Sun, Feb 15, 3:38 PM
F19083340: D21291.id50703.diff
Dec 2 2025, 12:34 PM
F19081992: D21291.id50702.diff
Dec 2 2025, 9:09 AM
F19041111: D21291.diff
Nov 26 2025, 2:40 PM
F19031344: D21291.diff
Nov 25 2025, 12:13 AM
F19015748: D21291.diff
Nov 23 2025, 1:19 AM
F19002192: D21291.diff
Nov 21 2025, 5:12 AM
F18985496: D21291.id50702.diff
Nov 17 2025, 1:04 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.