Skip to content

Commit

Permalink
Tweak ESLint config
Browse files Browse the repository at this point in the history
  • Loading branch information
axelboc committed Jun 14, 2022
1 parent d7de1aa commit 46f6917
Showing 1 changed file with 14 additions and 12 deletions.
26 changes: 14 additions & 12 deletions .eslintrc.cjs → .eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,33 +1,35 @@
const { createConfig } = require('eslint-config-galex/dist/createConfig')
const { files: jestFiles } = require('eslint-config-galex/dist/overrides/jest')
const { getDependencies } = require('eslint-config-galex/dist/getDependencies')
const {
files: reactFiles,
createJestOverride,
} = require('eslint-config-galex/dist/overrides/jest')
const {
createReactOverride,
} = require('eslint-config-galex/dist/overrides/react')

const dependencies = getDependencies()

module.exports = createConfig({
env: {
es6: true,
browser: true,
},
env: { es6: true },
overrides: [
{
files: reactFiles,
createReactOverride({
...dependencies,
rules: {
'import/no-default-export': 'off', // default exports are common in React
'unicorn/prefer-object-from-entries': 'off',
'unicorn/new-for-builtins': 'off',
'unicorn/prefer-set-has': 'off',
'sonarjs/no-duplicate-string': 'off',
},
},
{
files: jestFiles,
}),
createJestOverride({
...dependencies,
rules: {
'jest/no-focused-tests': 'warn', // warning instead of error
'jest/prefer-strict-equal': 'off', // `toEqual` is shorter and sufficient in most cases
'jest-formatting/padding-around-all': 'off', // allow writing concise two-line tests
'jest/require-top-level-describe': 'off', // filename should already be meaningful, extra nesting is unnecessary
},
},
}),
],
})

0 comments on commit 46f6917

Please sign in to comment.