topaLE
/
ghostfolio
mirror of https://github.com/ghostfolio/ghostfolio
1
0
Fork 0
Code Issues Projects Releases Wiki Activity
Browse Source

Task/migrate ESLint configuration to flat config without FlatCompat

pull/7938/head
David Requeno 5 days ago
parent
8b5ae8917e
commit
197f28e058
4 changed files with 158 additions and 167 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Unified View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 4
      apps/client/src/app/pages/portfolio/allocations/allocations-page.component.ts
  2. 291
      eslint.config.cjs
  3. 27
      package-lock.json
  4. 3
      package.json

4
apps/client/src/app/pages/portfolio/allocations/allocations-page.component.ts
View File

@ -398,10 +398,10 @@ export class GfAllocationsPageComponent implements OnInit {
this.holdings[assetProfileIdentifier] = { this.holdings[assetProfileIdentifier] = {
assetClass: assetClass:
position.assetProfile.assetClass || (UNKNOWN_KEY as AssetClass), position.assetProfile.assetClass ?? (UNKNOWN_KEY as AssetClass),
assetClassLabel: position.assetProfile.assetClassLabel ?? UNKNOWN_KEY, assetClassLabel: position.assetProfile.assetClassLabel ?? UNKNOWN_KEY,
assetSubClass: assetSubClass:
position.assetProfile.assetSubClass || (UNKNOWN_KEY as AssetSubClass), position.assetProfile.assetSubClass ?? (UNKNOWN_KEY as AssetSubClass),
assetSubClassLabel: assetSubClassLabel:
position.assetProfile.assetSubClassLabel ?? UNKNOWN_KEY, position.assetProfile.assetSubClassLabel ?? UNKNOWN_KEY,
currency: this.extractCurrency(position.assetProfile), currency: this.extractCurrency(position.assetProfile),

291
eslint.config.cjs
View File

@ -1,19 +1,14 @@
const { FlatCompat } = require('@eslint/eslintrc');
const js = require('@eslint/js');
const nxEslintPlugin = require('@nx/eslint-plugin'); const nxEslintPlugin = require('@nx/eslint-plugin');
const importPlugin = require('eslint-plugin-import');
const storybook = require('eslint-plugin-storybook'); const storybook = require('eslint-plugin-storybook');
const typescriptEslint = require('typescript-eslint');
const compat = new FlatCompat({
baseDirectory: __dirname,
recommendedConfig: js.configs.recommended
});
module.exports = [ module.exports = [
{ {
ignores: ['**/dist'] ignores: ['**/dist']
}, },
...storybook.configs['flat/recommended'], ...storybook.configs['flat/recommended'],
{ plugins: { '@nx': nxEslintPlugin } }, { plugins: { '@nx': nxEslintPlugin, import: importPlugin } },
{ {
files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'], files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
rules: { rules: {
@ -35,160 +30,134 @@ module.exports = [
'no-extra-semi': 'off' 'no-extra-semi': 'off'
} }
}, },
...compat ...nxEslintPlugin.configs['flat/typescript'],
.config({ ...nxEslintPlugin.configs['flat/javascript'],
extends: ['plugin:@nx/typescript'] ...typescriptEslint.config({
}) files: ['**/*.ts'],
.map((config) => ({ extends: [
...config, typescriptEslint.configs.recommendedTypeChecked,
files: ['**/*.ts', '**/*.tsx', '**/*.cts', '**/*.mts'], typescriptEslint.configs.stylisticTypeChecked
rules: { ],
...config.rules rules: {
} '@typescript-eslint/consistent-indexed-object-style': 'off',
})), '@typescript-eslint/dot-notation': 'off',
...compat '@typescript-eslint/explicit-member-accessibility': [
.config({ 'off',
extends: ['plugin:@nx/javascript'] {
}) accessibility: 'explicit'
.map((config) => ({ }
...config, ],
files: ['**/*.js', '**/*.jsx', '**/*.cjs', '**/*.mjs'], '@typescript-eslint/member-ordering': 'warn',
rules: { '@typescript-eslint/naming-convention': [
...config.rules 'off',
} {
})), selector: 'default',
...compat format: ['camelCase'],
.config({ leadingUnderscore: 'allow',
plugins: ['eslint-plugin-import', '@typescript-eslint'], trailingUnderscore: 'allow'
extends: [ },
'plugin:@typescript-eslint/recommended-type-checked', {
'plugin:@typescript-eslint/stylistic-type-checked' selector: ['variable', 'classProperty', 'typeProperty'],
] format: ['camelCase', 'UPPER_CASE'],
}) leadingUnderscore: 'allow',
.map((config) => ({ trailingUnderscore: 'allow'
...config, },
files: ['**/*.ts'], {
rules: { selector: 'objectLiteralProperty',
...config.rules, format: null
'@typescript-eslint/consistent-indexed-object-style': 'off', },
'@typescript-eslint/dot-notation': 'off', {
'@typescript-eslint/explicit-member-accessibility': [ selector: 'enumMember',
'off', format: ['camelCase', 'UPPER_CASE', 'PascalCase']
{ },
accessibility: 'explicit' {
} selector: 'typeLike',
], format: ['PascalCase']
'@typescript-eslint/member-ordering': 'warn', }
'@typescript-eslint/naming-convention': [ ],
'off', '@typescript-eslint/no-empty-interface': 'warn',
{ '@typescript-eslint/no-inferrable-types': [
selector: 'default', 'warn',
format: ['camelCase'], {
leadingUnderscore: 'allow', ignoreParameters: true
trailingUnderscore: 'allow' }
}, ],
{ '@typescript-eslint/no-non-null-assertion': 'warn',
selector: ['variable', 'classProperty', 'typeProperty'], '@typescript-eslint/no-shadow': [
format: ['camelCase', 'UPPER_CASE'], 'warn',
leadingUnderscore: 'allow', {
trailingUnderscore: 'allow' hoist: 'all'
}, }
{ ],
selector: 'objectLiteralProperty', '@typescript-eslint/unified-signatures': 'error',
format: null '@typescript-eslint/no-loss-of-precision': 'warn',
}, '@typescript-eslint/no-var-requires': 'warn',
{ 'arrow-body-style': 'off',
selector: 'enumMember', 'constructor-super': 'error',
format: ['camelCase', 'UPPER_CASE', 'PascalCase'] eqeqeq: ['error', 'smart'],
}, 'guard-for-in': 'warn',
{ 'id-blacklist': 'off',
selector: 'typeLike', 'id-match': 'off',
format: ['PascalCase'] 'import/no-deprecated': 'warn',
} 'no-bitwise': 'error',
], 'no-caller': 'error',
'@typescript-eslint/no-empty-interface': 'warn', 'no-debugger': 'error',
'@typescript-eslint/no-inferrable-types': [ 'no-empty': 'off',
'warn', 'no-eval': 'error',
{ 'no-fallthrough': 'error',
ignoreParameters: true 'no-new-wrappers': 'error',
} 'no-restricted-imports': ['error', 'rxjs/Rx'],
], 'no-undef-init': 'error',
'@typescript-eslint/no-non-null-assertion': 'warn', 'no-underscore-dangle': 'off',
'@typescript-eslint/no-shadow': [ 'no-var': 'error',
'warn', radix: 'error',
{ 'no-unsafe-optional-chaining': 'warn',
hoist: 'all' 'no-extra-boolean-cast': 'warn',
} 'no-empty-pattern': 'warn',
], 'no-useless-catch': 'warn',
'@typescript-eslint/unified-signatures': 'error', 'no-unsafe-finally': 'warn',
'@typescript-eslint/no-loss-of-precision': 'warn', 'no-prototype-builtins': 'warn',
'@typescript-eslint/no-var-requires': 'warn', 'no-async-promise-executor': 'warn',
'arrow-body-style': 'off', 'no-constant-condition': 'warn',
'constructor-super': 'error',
eqeqeq: ['error', 'smart'],
'guard-for-in': 'warn',
'id-blacklist': 'off',
'id-match': 'off',
'import/no-deprecated': 'warn',
'no-bitwise': 'error',
'no-caller': 'error',
'no-debugger': 'error',
'no-empty': 'off',
'no-eval': 'error',
'no-fallthrough': 'error',
'no-new-wrappers': 'error',
'no-restricted-imports': ['error', 'rxjs/Rx'],
'no-undef-init': 'error',
'no-underscore-dangle': 'off',
'no-var': 'error',
radix: 'error',
'no-unsafe-optional-chaining': 'warn',
'no-extra-boolean-cast': 'warn',
'no-empty-pattern': 'warn',
'no-useless-catch': 'warn',
'no-unsafe-finally': 'warn',
'no-prototype-builtins': 'warn',
'no-async-promise-executor': 'warn',
'no-constant-condition': 'warn',
// The following rules are part of eslint:recommended // The following rules are part of eslint:recommended
// and can be remove once solved // and can be remove once solved
'no-loss-of-precision': 'warn', 'no-loss-of-precision': 'warn',
// The following rules are part of @typescript-eslint/recommended-type-checked // The following rules are part of @typescript-eslint/recommended-type-checked
// and can be remove once solved // and can be remove once solved
'@typescript-eslint/await-thenable': 'warn', '@typescript-eslint/await-thenable': 'warn',
'@typescript-eslint/ban-ts-comment': 'warn', '@typescript-eslint/ban-ts-comment': 'warn',
'@typescript-eslint/no-base-to-string': 'warn', '@typescript-eslint/no-base-to-string': 'warn',
'@typescript-eslint/no-empty-object-type': 'warn', '@typescript-eslint/no-empty-object-type': 'warn',
'@typescript-eslint/no-explicit-any': 'warn', '@typescript-eslint/no-explicit-any': 'warn',
'@typescript-eslint/no-floating-promises': 'warn', '@typescript-eslint/no-floating-promises': 'warn',
'@typescript-eslint/no-misused-promises': 'warn', '@typescript-eslint/no-misused-promises': 'warn',
'@typescript-eslint/no-redundant-type-constituents': 'warn', '@typescript-eslint/no-redundant-type-constituents': 'warn',
'@typescript-eslint/no-require-imports': 'warn', '@typescript-eslint/no-require-imports': 'warn',
'@typescript-eslint/no-unnecessary-type-assertion': 'warn', '@typescript-eslint/no-unnecessary-type-assertion': 'warn',
'@typescript-eslint/no-unsafe-argument': 'warn', '@typescript-eslint/no-unsafe-argument': 'warn',
'@typescript-eslint/no-unsafe-assignment': 'warn', '@typescript-eslint/no-unsafe-assignment': 'warn',
'@typescript-eslint/no-unsafe-enum-comparison': 'warn', '@typescript-eslint/no-unsafe-enum-comparison': 'warn',
'@typescript-eslint/no-unsafe-member-access': 'warn', '@typescript-eslint/no-unsafe-member-access': 'warn',
'@typescript-eslint/no-unsafe-return': 'warn', '@typescript-eslint/no-unsafe-return': 'warn',
'@typescript-eslint/no-unsafe-call': 'warn', '@typescript-eslint/no-unsafe-call': 'warn',
'@typescript-eslint/no-unused-vars': [ '@typescript-eslint/no-unused-vars': [
'error', 'error',
{ {
caughtErrors: 'none' caughtErrors: 'none'
} }
], ],
'@typescript-eslint/no-wrapper-object-types': 'warn', '@typescript-eslint/no-wrapper-object-types': 'warn',
'@typescript-eslint/only-throw-error': 'warn', '@typescript-eslint/only-throw-error': 'warn',
'@typescript-eslint/prefer-promise-reject-errors': 'warn', '@typescript-eslint/prefer-promise-reject-errors': 'warn',
'@typescript-eslint/require-await': 'warn', '@typescript-eslint/require-await': 'warn',
'@typescript-eslint/restrict-template-expressions': 'warn', '@typescript-eslint/restrict-template-expressions': 'warn',
'@typescript-eslint/unbound-method': 'warn', '@typescript-eslint/unbound-method': 'warn',
// The following rules are part of @typescript-eslint/stylistic-type-checked // The following rules are part of @typescript-eslint/stylistic-type-checked
// and can be remove once solved // and can be remove once solved
'@typescript-eslint/prefer-nullish-coalescing': 'warn' // TODO: Requires strictNullChecks: true '@typescript-eslint/prefer-nullish-coalescing': 'warn' // TODO: Requires strictNullChecks: true
} }
})) })
]; ];

27
package-lock.json
View File

@ -122,8 +122,6 @@
"@angular/language-service": "22.1.2", "@angular/language-service": "22.1.2",
"@angular/localize": "22.1.2", "@angular/localize": "22.1.2",
"@angular/pwa": "22.1.4", "@angular/pwa": "22.1.4",
"@eslint/eslintrc": "3.3.1",
"@eslint/js": "9.35.0",
"@nestjs/schematics": "11.1.0", "@nestjs/schematics": "11.1.0",
"@nestjs/testing": "11.2.3", "@nestjs/testing": "11.2.3",
"@nx/angular": "23.1.1", "@nx/angular": "23.1.1",
@ -177,6 +175,7 @@
"ts-node": "10.9.2", "ts-node": "10.9.2",
"tslib": "2.8.1", "tslib": "2.8.1",
"typescript": "6.0.3", "typescript": "6.0.3",
"typescript-eslint": "8.67.0",
"webpack-cli": "7.1.0" "webpack-cli": "7.1.0"
}, },
"engines": { "engines": {
@ -34881,6 +34880,30 @@
"node": ">=14.17" "node": ">=14.17"
} }
}, },
"node_modules/typescript-eslint": {
"version": "8.67.0",
"resolved": "https://registry.npmjs.org/typescript-eslint/-/typescript-eslint-8.67.0.tgz",
"integrity": "sha512-S2udFs8tCKEKffuJ4TB1idGUZiXdCPGi3IPBGWXarbLQ5UPXORV8QEVzJ4gCRduURMb5EkpNCdjbk0eDIuI8Yg==",
"dev": true,
"license": "MIT",
"dependencies": {
"@typescript-eslint/eslint-plugin": "8.67.0",
"@typescript-eslint/parser": "8.67.0",
"@typescript-eslint/typescript-estree": "8.67.0",
"@typescript-eslint/utils": "8.67.0"
},
"engines": {
"node": "^18.18.0 || ^20.9.0 || >=21.1.0"
},
"funding": {
"type": "opencollective",
"url": "https://opencollective.com/typescript-eslint"
},
"peerDependencies": {
"eslint": "^8.57.0 || ^9.0.0 || ^10.0.0",
"typescript": ">=4.8.4 <6.1.0"
}
},
"node_modules/uglify-js": { "node_modules/uglify-js": {
"version": "3.19.3", "version": "3.19.3",
"resolved": "https://registry.npmjs.org/uglify-js/-/uglify-js-3.19.3.tgz", "resolved": "https://registry.npmjs.org/uglify-js/-/uglify-js-3.19.3.tgz",

3
package.json
View File

@ -166,8 +166,6 @@
"@angular/language-service": "22.1.2", "@angular/language-service": "22.1.2",
"@angular/localize": "22.1.2", "@angular/localize": "22.1.2",
"@angular/pwa": "22.1.4", "@angular/pwa": "22.1.4",
"@eslint/eslintrc": "3.3.1",
"@eslint/js": "9.35.0",
"@nestjs/schematics": "11.1.0", "@nestjs/schematics": "11.1.0",
"@nestjs/testing": "11.2.3", "@nestjs/testing": "11.2.3",
"@nx/angular": "23.1.1", "@nx/angular": "23.1.1",
@ -221,6 +219,7 @@
"ts-node": "10.9.2", "ts-node": "10.9.2",
"tslib": "2.8.1", "tslib": "2.8.1",
"typescript": "6.0.3", "typescript": "6.0.3",
"typescript-eslint": "8.67.0",
"webpack-cli": "7.1.0" "webpack-cli": "7.1.0"
}, },
"engines": { "engines": {

Write Preview
Loading…
Cancel
Save