Skip to content

Commit a5d5b5f

Browse files
committed
Refactor setting of default value
1 parent f7daf13 commit a5d5b5f

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

tools/make/common.mk

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,11 @@ KEYWORDS ?= 'TODO|FIXME|WARNING|HACK|NOTE|OPTIMIZE'
4040
ifndef FAST_FAIL
4141
FAIL_FAST := true
4242
else
43+
ifeq ($(FAST_FAIL), 0)
4344
FAIL_FAST := false
45+
else
46+
FAIL_FAST := true
47+
endif
4448
endif
4549

4650

tools/make/lib/lint/javascript/eslint.mk

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,11 +56,20 @@ ESLINT_FLAGS ?= \
5656
# This target lints only JavaScript source files.
5757

5858
eslint-src: $(NODE_MODULES)
59+
ifeq ($(FAIL_FAST), true)
5960
$(QUIET) $(FIND_SOURCES_CMD) | grep '^[\/]\|^[a-zA-Z]:[/\]' | while read -r file; do \
6061
echo ''; \
6162
echo "Linting file: $$file"; \
6263
$(ESLINT) $(ESLINT_FLAGS) --config $(ESLINT_CONF) $$file || exit 1; \
6364
done
65+
else
66+
$(QUIET) EXIT_CODE=0 && $(FIND_SOURCES_CMD) | grep '^[\/]\|^[a-zA-Z]:[/\]' | while read -r file; do \
67+
echo ''; \
68+
echo "Linting file: $$file"; \
69+
$(ESLINT) $(ESLINT_FLAGS) --config $(ESLINT_CONF) $$file || { EXIT_CODE=1; exit 0; } \
70+
done
71+
exit $(EXIT_CODE)
72+
endif
6473

6574
.PHONY: eslint-src
6675

0 commit comments

Comments
 (0)