-
Notifications
You must be signed in to change notification settings - Fork 6
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
chore(deps): use @empathyco/eslint-config as code format config #514
Conversation
7ee3b77
to
b71e9f1
Compare
Check PR 514 preview 👀 |
Check PR 514 preview 👀 |
Check PR 514 preview 👀 |
Check PR 514 preview 👀 |
Check PR 514 preview 👀 |
eslint.config.mjs
Outdated
rules: { | ||
'vuejs-accessibility/no-autofocus': 'off', | ||
'vue/multi-word-component-names': 'off', | ||
'node/prefer-global/process': 'off', |
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.
I think we can remove this configuration. The only rule that seems we are violating is the node/prefer-global/process
and I'd just disable it on the plugin.options.ts file
Check PR 514 preview 👀 |
Pull request template
Pull Request Blocked by #513
Change legacy
@empathyco/eslint-plugin-x
by the new@empathyco/eslint-config
Motivation and context
Type of change
What is the destination branch of this PR?
Main
How has this been tested?
Tests performed according to testing guidelines:
Checklist: