Skip to content

Commit 81cc729

Browse files
author
Aviv Keller
authored
build: fix eslint makefile target
PR-URL: #54999 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
1 parent 8b8fc53 commit 81cc729

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Makefile

+4-4
Original file line numberDiff line numberDiff line change
@@ -1388,19 +1388,19 @@ run-lint-js = tools/eslint/node_modules/eslint/bin/eslint.js --cache \
13881388
--max-warnings=0 --report-unused-disable-directives $(LINT_JS_TARGETS)
13891389
run-lint-js-fix = $(run-lint-js) --fix
13901390

1391-
tools/eslint/node_modules: tools/eslint/package-lock.json
1391+
tools/eslint/node_modules/eslint/bin/eslint.js: tools/eslint/package-lock.json
13921392
-cd tools/eslint && $(call available-node,$(run-npm-ci))
13931393

13941394
.PHONY: lint-js-fix
1395-
lint-js-fix: tools/eslint/node_modules
1395+
lint-js-fix: tools/eslint/node_modules/eslint/bin/eslint.js
13961396
@$(call available-node,$(run-lint-js-fix))
13971397

13981398
.PHONY: lint-js
13991399
.PHONY: lint-js-doc
14001400
# Note that on the CI `lint-js-ci` is run instead.
14011401
# Lints the JavaScript code with eslint.
14021402
lint-js-doc: LINT_JS_TARGETS=doc
1403-
lint-js lint-js-doc: tools/eslint/node_modules
1403+
lint-js lint-js-doc: tools/eslint/node_modules/eslint/bin/eslint.js
14041404
@if [ "$(shell $(node_use_openssl))" != "true" ]; then \
14051405
echo "Skipping $@ (no crypto)"; \
14061406
else \
@@ -1417,7 +1417,7 @@ run-lint-js-ci = tools/eslint/node_modules/eslint/bin/eslint.js \
14171417

14181418
.PHONY: lint-js-ci
14191419
# On the CI the output is emitted in the TAP format.
1420-
lint-js-ci: tools/eslint/node_modules
1420+
lint-js-ci: tools/eslint/node_modules/eslint/bin/eslint.js
14211421
$(info Running JS linter...)
14221422
@$(call available-node,$(run-lint-js-ci))
14231423

0 commit comments

Comments
 (0)