github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-07 20:31:06 -05:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #18656 from overleaf/jpa-fix-double-lint

Browse source 
[web] eslint with multiple --ignore-pattern is broken

GitOrigin-RevId: eef4bdd2e43c6f576ee5997f8e9a2bd9f385ab48
This commit is contained in:
Jakob Ackermann 2024-06-03 10:36:42 +02:00 committed by Copybot
parent 9bed2dc73e
commit 167bb66939
1 changed files with 3 additions and 50 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

53
services/web/Makefile
View file

@ -393,56 +393,9 @@ WITH_NODE_MODULES_PATH = \
$(WITH_NODE_MODULES_PATH): export PATH=$(NODE_MODULES_PATH)
lint: lint_backend
lint_backend:
npx eslint \
'app.{js,jsx,mjs,ts,tsx}' \
'app/**/*.{js,jsx,mjs,ts,tsx}' \
'modules/*/index.{js,jsx,mjs,ts,tsx}' \
'modules/*/app/**/*.{js,jsx,mjs,ts,tsx}' \
--max-warnings=0
lint: lint_frontend
lint_frontend:
npx eslint \
'frontend/**/*.{js,jsx,mjs,ts,tsx}' \
'modules/**/frontend/**/*.{js,jsx,mjs,ts,tsx}' \
--max-warnings=0
lint: lint_test
lint_test: lint_test_app
lint_test_app: lint_test_app_unit
lint_test_app_unit:
npx eslint \
'test/unit/**/*.{js,jsx,mjs,ts,tsx}' \
--max-warnings=0
lint_test_app: lint_test_app_rest
lint_test_app_rest:
npx eslint \
'test/**/*.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'test/unit/**/*.{js,jsx,mjs,ts,tsx}' \
--max-warnings=0
lint_test: lint_test_modules
lint_test_modules:
npx eslint \
'modules/*/test/**/*.{js,jsx,mjs,ts,tsx}' \
--max-warnings=0
lint: lint_misc
# migrations, scripts, webpack config
lint_misc:
npx eslint . \
--ignore-pattern 'app.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'app/**/*.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'modules/*/app/**/*.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'modules/*/index.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'frontend/**/*.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'modules/**/frontend/**/*.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'test/**/*.{js,jsx,mjs,ts,tsx}' \
--ignore-pattern 'modules/*/test/**/*.{js,jsx,mjs,ts,tsx}' \
--max-warnings=0
lint: lint_eslint
lint_eslint:
npm run lint
lint: lint_pug
lint_pug: