1466902 - [mozlint] Ignore untracked files

Status: NEW

(Developer Infrastructure :: Lint and Formatting, enhancement)

Description

[Andrew Halberstadt [:ahal]]

We never want to lint untracked files. This is a little difficult to implement because the underlying linters don't normally know anything about version control.

For example, if I run |mach lint browser|, we forward the "browser" path wholesale to e.g, eslint. ESlint will do its own filtering and lint all relevant files under /browser. Checking for untracked files would mean expanding the "/browser" directory to a recursive list of every file it contains, then filtering out all of the files that aren't tracked. This would likely be prohibitively slow.

I'm not really sure how to solve this efficiently, but it's at least worth having on file.

Comment 1

[Andrew Halberstadt [:ahal]]

Actually, it would be a lot easier to filter out the results of untracked files after the fact (the set of files containing errors should be much smaller than the set of files being linted). The downside to doing this is that we'd still be linting things unnecessarily, but at least they wouldn't be showing up as errors.