Skip to content

Commit 0291be5

Browse files
Xstoudiaduh95
authored andcommitted
doc: test and test-only targets do not run linter
`make test` and `make test-only` targets no longer runs linters. This commit updates the inline comments and building doc to reflect reality. Fixes: #58606 PR-URL: #62120 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Richard Lau <richard.lau@ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: René <contact.9a5d6388@renegade334.me.uk> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
1 parent 70f8057 commit 0291be5

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

BUILDING.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -324,11 +324,9 @@ If you are running tests before submitting a pull request, use:
324324
make -j4 test
325325
```
326326

327-
`make -j4 test` does a full check on the codebase, including running linters and
328-
documentation tests.
327+
`make -j4 test` does a full check on the codebase, including documentation tests.
329328

330-
To run the linter without running tests, use
331-
`make lint`/`vcbuild lint`. It will lint JavaScript, C++, and Markdown files.
329+
To run the linter, use `make lint`/`vcbuild lint`. It will lint JavaScript, C++, and Markdown files.
332330

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

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,7 @@ coverage-run-js: ## Run JavaScript tests with coverage.
338338

339339
.PHONY: test
340340
# This does not run tests of third-party libraries inside deps.
341-
test: all ## Run default tests, linters, and build docs.
341+
test: all ## Run default tests and build docs.
342342
$(MAKE) -s tooltest
343343
$(MAKE) -s test-doc
344344
$(MAKE) -s build-addons
@@ -349,7 +349,7 @@ test: all ## Run default tests, linters, and build docs.
349349
$(MAKE) -s jstest
350350

351351
.PHONY: test-only
352-
test-only: all ## Run default tests, without linters or building the docs.
352+
test-only: all ## Run default tests without building the docs.
353353
$(MAKE) build-addons
354354
$(MAKE) build-js-native-api-tests
355355
$(MAKE) build-node-api-tests

0 commit comments

Comments
 (0)