Skip to content

Commit 662db12

Browse files
Simplify linting logic in eslint.mk
Signed-off-by: Suyash Pathak <[email protected]>
1 parent 3ee6f99 commit 662db12

File tree

1 file changed

+7
-15
lines changed

1 file changed

+7
-15
lines changed

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

Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -253,26 +253,18 @@ endif
253253
eslint-files: $(NODE_MODULES)
254254
ifeq ($(FAIL_FAST), true)
255255
$(QUIET) for file in $(FILES); do \
256-
if [ -f "$$file" ]; then \
257256
echo ''; \
258-
echo "Linting file: $$file"; \
259-
$(ESLINT) $(eslint_flags) --config $(ESLINT_CONF) $$file || exit 1; \
260-
else \
261-
echo "Skipping missing file: $$file"; \
262-
fi; \
257+
echo "Linting file: $$file"; \
258+
$(ESLINT) $(eslint_flags) --config $(ESLINT_CONF) $$file || exit 1; \
263259
done
264260
else
265261
$(QUIET) status=0; \
266262
for file in $(FILES); do \
267-
if [ -f "$$file" ]; then \
268-
echo ''; \
269-
echo "Linting file: $$file"; \
270-
if ! $(ESLINT) $(eslint_flags) --config $(ESLINT_CONF) $$file; then \
271-
echo 'Linting failed.'; \
272-
status=1; \
273-
fi; \
274-
else \
275-
echo "Skipping missing file: $$file"; \
263+
echo ''; \
264+
echo "Linting file: $$file"; \
265+
if ! $(ESLINT) $(eslint_flags) --config $(ESLINT_CONF) $$file; then \
266+
echo 'Linting failed.'; \
267+
status=1; \
276268
fi; \
277269
done; \
278270
exit $$status;

0 commit comments

Comments
 (0)