fix(eslint): ignore python venv + playwright artifacts
The root 'Build, Test & Typecheck' CI job was failing at lint with:
services/extraction-service/python/.venv/lib/python3.13/site-packages/
urllib3/contrib/emscripten/emscripten_fetch_worker.js
10:21 error 'TextEncoder' is not defined no-undef
12:1 error 'self' is not defined no-undef
40:9 error 'console' is not defined no-undef
70:30 error 'fetch' is not defined no-undef
...
urllib3 ships emscripten worker stubs that ESLint should never see.
The Python venv was added recently and the root eslint.config.js
didn't yet have any '**/.venv/**' / '**/__pycache__/**' ignore
patterns, so 'pnpm -r exec eslint . --ext .ts,.tsx' picked them up
when run from the extraction-service workspace.
Added four ignore patterns to the root config:
'**/.venv/**'
'**/venv/**'
'**/__pycache__/**'
'**/playwright-report/**'
'**/test-results/**'
The Python venv globs are the actual fix; the playwright ones are
preventative — per-package configs already cover most cases but
the root config is what 'pnpm -r exec eslint' walks.
This commit is contained in:
parent
5da706ead1
commit
9be49acbfe
@ -12,6 +12,17 @@ export default [
|
||||
'*.config.ts',
|
||||
'**/dist/**',
|
||||
'__LOCAL_LLMs/**',
|
||||
// Python virtual env in extraction-service/python ships with
|
||||
// urllib3's emscripten worker stubs (JS files that reference
|
||||
// browser globals like `self`, `fetch`, `TextEncoder`). Those
|
||||
// files are NOT project source and must never be linted.
|
||||
'**/.venv/**',
|
||||
'**/venv/**',
|
||||
'**/__pycache__/**',
|
||||
// Playwright artifacts (per-package configs already exclude
|
||||
// these, but the root config is what runs in CI).
|
||||
'**/playwright-report/**',
|
||||
'**/test-results/**',
|
||||
],
|
||||
},
|
||||
js.configs.recommended,
|
||||
|
||||
Loading…
Reference in New Issue
Block a user