Skip to content

Commit

Permalink
pdf: fix migrate eslint config
Browse files Browse the repository at this point in the history
The . after the eslint command is important, else it lints nothing.
Explicitly exclude the generated files, ignoring it via the .gitignored files did not work.

Issue: ecamp#5282
  • Loading branch information
BacLuc committed Aug 13, 2024
1 parent cabfbdd commit 9abed89
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions pdf/eslint.config.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@ export default [
'plugin:prettier/recommended',
'@vue/eslint-config-prettier'
),
{
ignores: ['dist/*.mjs'],
},

includeIgnoreFile(gitignorePath),

Expand All @@ -36,9 +39,6 @@ export default [
...globals.jest,
},

ecmaVersion: 5,
sourceType: 'commonjs',

parserOptions: {
parser: '@babel/eslint-parser',
},
Expand Down
4 changes: 2 additions & 2 deletions pdf/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,10 @@
"preview": "vite preview",
"test:unit": "vitest --coverage",
"lint": "npm run lint:eslint && npm run lint:prettier",
"lint:eslint": "eslint --fix",
"lint:eslint": "eslint --fix .",
"lint:prettier": "prettier --write --ignore-path .prettierignore **/*.{css,scss,json,mjs}",
"lint:check": "npm run lint:check:eslint && npm run lint:check:prettier",
"lint:check:eslint": "eslint",
"lint:check:eslint": "eslint .",
"lint:check:prettier": "prettier --check --ignore-path .prettierignore **/*.{css,scss,json,mjs}"
},
"dependencies": {
Expand Down

0 comments on commit 9abed89

Please sign in to comment.