-
Notifications
You must be signed in to change notification settings - Fork 80
Closed
Description
macos is super eager to not pass commands to the shell at all, if it thinks they're simple.
Thus the PATH logic doesn't work for things like tsc, or some invocations of eslint. Yes, some.
There are at least three ways to fix these:
lint-1:
@: ; eslint --config $(ROOT)/eslint_test.json --max-warnings 0 test/
lint-2:
@eslint --config $(ROOT)/eslint_test.* --max-warnings 0 test/
lint-3:
@../.node_modules/.bin/eslint --config $(ROOT)/eslint_test.json --max-warnings 0 test/
common.mk could probably also define ESLINT and TSC to the path in lint-3, which would double-fix the issue, probably, as the shell variable would trigger the shell, and it'd be a real path.
@stasm, opinions?
Metadata
Metadata
Assignees
Labels
No labels