Skip to content

Commit

Permalink
doc: add lint-js-fix into BUILDING.md
Browse files Browse the repository at this point in the history
When there is any auto-fixable linting errors, there is not a way to
pass `--fix` into `make lint-js` as far as I am aware of. Spent quiet
some time fixing them one by one and just realised `lint-js-fix` exists.

PR-URL: #52290
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
  • Loading branch information
jakecastelli authored Apr 10, 2024
1 parent 411503b commit d32a914
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions BUILDING.md
Original file line number Diff line number Diff line change
Expand Up @@ -327,6 +327,8 @@ documentation tests.
To run the linter without running tests, use
`make lint`/`vcbuild lint`. It will lint JavaScript, C++, and Markdown files.

To fix auto fixable JavaScript linting errors, use `make lint-js-fix`.

If you are updating tests and want to run tests in a single test file
(e.g. `test/parallel/test-stream2-transform.js`):

Expand Down

0 comments on commit d32a914

Please sign in to comment.