Skip to content

tools: don't lint files that have not changed

Rodrigo Muino Tomonari requested to merge github/fork/joyeecheung/less-lint into master
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

After https://github.com/nodejs/node/pull/16284 landed we now run linters before running test. This PR prevents make from linting files that have not changed, so e.g. if someone only touches lib they don't have to wait for cpp linter or the markdown linter to finish.

JS linter does not need this because eslint has cache already.

The time for running make lint the second time before this patch:

make lint  37.86s user 2.08s system 105% cpu 37.893 total

After:

make lint  2.57s user 0.56s system 103% cpu 3.032 total

Merge request reports