-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix issue with outside rules and indent rule #15
Fix issue with outside rules and indent rule #15
Conversation
"jsx-quotes": ["error", "prefer-single"], | ||
"no-confusing-arrow": ["error", { allowParens: false }], | ||
"no-tabs": ["error", { allowIndentationTabs: true }], | ||
indent: ["error", 4], |
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.
Since these rules are based on migrating prettier to eslint: #12, assigned ident
rule to have a tab width of 4
based on prettierrc
"error", | ||
"single", | ||
{ avoidEscape: true, allowTemplateLiterals: false }, | ||
], |
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 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.
good catch 👍
@@ -37,7 +37,7 @@ | |||
"eslint-plugin-jest": "^27.4.2", | |||
"eslint-plugin-jest-dom": "^5.1.0", | |||
"eslint-plugin-jest-playwright": "^0.9.0", | |||
"eslint-plugin-prettier": "^3.3.1", | |||
"eslint-plugin-prettier": "^5.0.0", |
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.
Upgraded eslint-plugin-prettier to v5 to resolve error based on this issue
Fixed an issue where some prettier rules are outside the
rules
attribute, andident
has invalid argumentfalse