chore(eslint): convert eslint config to flat config#376
Conversation
csouchet
added
the
chore
|
♻️ PR Preview c1d054b has been successfully destroyed since this PR has been closed.
🤖 By surge-preview |
csouchet
force-pushed
the
chore/convert_eslint_config_to_flat
branch
from
079d073 to
55ed97c
Compare
csouchet
force-pushed
the
chore/convert_eslint_config_to_flat
branch
from
55ed97c to
4ad406d
Compare
|
# Conflicts: # package-lock.json # package.json
tbouffard
left a comment
tbouffard
left a comment
There was a problem hiding this comment.
There are some additional linting rules which are added whereas we only want to switch the configuration.
Review the changes and if we want to apply the new rules, apply them to the main branch first.
| this.model = bpmnVisualization.graph.model; | ||
| } | ||
|
|
||
| /* eslint-disable jest/no-standalone-expect -- util code, including expect calls */ |
There was a problem hiding this comment.
question: why is this removed?
# Conflicts: # .eslintrc.cjs # package-lock.json # package.json
…uration is at the project root
jest: ensure recommended and style rules are enabled + apply jsdom config correctly test files: - use right glob - centralize config remove extra unicorn configuration that didn't exist in the former configuration The extra configuration came from bpmn-visualization and had been copy/paste here by mistake
# Conflicts: # package-lock.json # packages/addons/package.json
|
tbouffard
left a comment
tbouffard
left a comment
There was a problem hiding this comment.
LGTM, all plugins have been tested individually: they detect and can auto-fix errors/warnings.
2 participants
No description provided.