-
Notifications
You must be signed in to change notification settings - Fork 340
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
Various review feedback for ESLint configuration #204
Conversation
e3534fc
to
f30e5d2
Compare
.prettierrc
Outdated
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've removed the Prettier config, since the default should be good enough. This leads to some code formatting changes, but nothing significant. Also the singleQuote
option seems to have no impact anyways, presumably as this is already overwritten at the ESLint level.
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.
Let's not use 80 chars for printwidth and set it back to 100.
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.
Sure, added a commit to restore this.
Co-authored-by: Frederik Prijck <[email protected]>
This PR contains some changes for the ESLint configuration based on what my review for #200 would have been.