-
Notifications
You must be signed in to change notification settings - Fork 272
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deps: Update eslint to 9.3.0 #578
Merged
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
3a6fbee
deps: Install eslint 9
emil-litwiniec 2cae999
deps: Rewrite eslint config in webapp-core
emil-litwiniec ba93d01
deps: Config rewrite webapp-crud-demo
emil-litwiniec 3e7c39f
deps: Config rewrite webapp-contentful
emil-litwiniec a268d08
deps: Config rewrite webapp-api-client
emil-litwiniec c74e365
deps: Config rewrite webapp-documents
emil-litwiniec 7a2f581
deps: Config rewrite webapp-emails
emil-litwiniec 3520d95
deps: Config rewrite webapp-finances
emil-litwiniec 41add45
deps: Config rewrite webapp-generative-ai
emil-litwiniec 3a45789
deps: Config rewrite webapp-notifications
emil-litwiniec 93645dd
deps: Config rewrite webapp-tenants
emil-litwiniec 7ac279b
deps: Config rewrite webapp
emil-litwiniec f79b48f
deps: Config rewrite internal/cli
emil-litwiniec 8bd3ed7
deps: Config rewrite status-dasboard
emil-litwiniec c71cb67
deps: Config rewrite docs
emil-litwiniec 99bd4bd
deps: Config rewrite tools
emil-litwiniec b987eb6
deps: Config rewrite infra-core
emil-litwiniec c33c596
deps: Config rewrite infra-functions
emil-litwiniec 1f2fdb1
deps: Config rewrite infra-shared
emil-litwiniec eb3ac22
deps: Config rewrite backend
emil-litwiniec b3978ff
chore: Use commonjs for eslint config files
emil-litwiniec eac478e
chore: Update generate template files
emil-litwiniec 65a88ad
fix: webapp eslint config redundant root import
emil-litwiniec ddaf840
fix: Missing hook deps from rule testing
emil-litwiniec 52e7d7a
fix: webapp-emails config error
emil-litwiniec 1132f59
deps: Config rewrite workers
emil-litwiniec b583a06
chore: Remove unused recc glob chars
emil-litwiniec 139b6bc
fix: workers lint bash script not seeing project root
emil-litwiniec 714ca7f
chore: Remove redundant empty rules config objects
emil-litwiniec d1d84b3
deps: Update eslint plugins
emil-litwiniec a3f8672
fix: react types divereged versions
emil-litwiniec ce2edba
chore: Remove empty rules objects
emil-litwiniec File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,92 @@ | ||
const { FlatCompat } = require('@eslint/eslintrc'); | ||
const nxEslintPlugin = require('@nx/eslint-plugin'); | ||
const js = require('@eslint/js'); | ||
|
||
const compat = new FlatCompat({ | ||
baseDirectory: __dirname, | ||
recommendedConfig: js.configs.recommended, | ||
}); | ||
|
||
module.exports = [ | ||
{ plugins: { '@nx': nxEslintPlugin } }, | ||
{ | ||
settings: { | ||
'import/parsers': { '@typescript-eslint/parser': ['.ts', '.tsx'] }, | ||
'import/resolver': { | ||
typescript: { | ||
alwaysTryTypes: true, | ||
project: 'tsconfig.base.json', | ||
}, | ||
}, | ||
}, | ||
}, | ||
{ | ||
files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'], | ||
rules: { | ||
'@nx/enforce-module-boundaries': [ | ||
'error', | ||
{ | ||
enforceBuildableLibDependency: true, | ||
allow: [ | ||
'@sb/webapp-core/**', | ||
'@sb/webapp-api-client/**', | ||
'@sb/webapp-contentful/**', | ||
'@sb/webapp-crud-demo/**', | ||
'@sb/webapp-documents/**', | ||
'@sb/webapp-finances/**', | ||
'@sb/webapp-generative-ai/**', | ||
'@sb/webapp-notifications/**', | ||
'@sb/webapp-tenants/**', | ||
], | ||
depConstraints: [ | ||
{ | ||
sourceTag: '*', | ||
onlyDependOnLibsWithTags: ['*'], | ||
}, | ||
], | ||
}, | ||
], | ||
}, | ||
}, | ||
...compat | ||
.config({ | ||
extends: [ | ||
'plugin:@nx/typescript', | ||
'prettier', | ||
'plugin:@typescript-eslint/recommended', | ||
'plugin:import/errors', | ||
'plugin:import/warnings', | ||
'plugin:import/typescript', | ||
], | ||
}) | ||
.map((config) => ({ | ||
...config, | ||
files: ['**/*.ts', '**/*.tsx'], | ||
rules: {}, | ||
})), | ||
...compat | ||
.config({ | ||
extends: [ | ||
'plugin:@nx/javascript', | ||
'prettier', | ||
'plugin:import/errors', | ||
'plugin:import/warnings', | ||
'plugin:import/typescript', | ||
], | ||
}) | ||
.map((config) => ({ | ||
...config, | ||
files: ['**/*.js', '**/*.jsx'], | ||
rules: {}, | ||
})), | ||
...compat.config({ env: { jest: true } }).map((config) => ({ | ||
...config, | ||
files: ['**/*.spec.ts', '**/*.spec.tsx', '**/*.spec.js', '**/*.spec.jsx'], | ||
rules: {}, | ||
})), | ||
...compat.config({ parser: 'jsonc-eslint-parser' }).map((config) => ({ | ||
...config, | ||
files: ['**/*.json'], | ||
rules: {}, | ||
})), | ||
]; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
const rootConfig = require('../../eslint.config.js'); | ||
|
||
module.exports = [ | ||
...rootConfig, | ||
{ | ||
ignores: [ | ||
'!**/*', | ||
'node_modules/**/*', | ||
'cdk.out/**/*', | ||
'__pypackages__/**/*', | ||
'apps/finances/static/djstripe/*', | ||
], | ||
}, | ||
]; |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
const rootConfig = require('../../../eslint.config.js'); | ||
|
||
module.exports = [ | ||
...rootConfig, | ||
{ | ||
ignores: [ | ||
'node_modules/**/*', | ||
'cdk.out/**/*', | ||
'src/lib/patterns/ecr-sync/**/*', | ||
], | ||
}, | ||
]; |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
const baseConfig = require('../../../eslint.config.js'); | ||
|
||
module.exports = [ | ||
...baseConfig, | ||
{ | ||
ignores: ['node_modules/**/*', '.serverless/**/*', 'cdk.out/**/*'], | ||
}, | ||
]; |
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is some weird dep that breaks the react types, which makes docs fail to build from time to time when new pnpm lock is being created. I need to investigate it and pin naughty package to a specific version.