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
Align prettier settings and eslint #49
Align prettier settings and eslint #49
Conversation
@@ -5,8 +5,9 @@ | |||
"extends": "standard", | |||
"rules": { | |||
"semi": [2, "always"], | |||
"indent": ["error", 4], | |||
"mocha/no-exclusive-tests": "error" | |||
"indent": 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.
Rule switched of since indent doesn't always agree with prettier decisions, particularly for very long lines.
"mocha/no-exclusive-tests": "error" | ||
"indent": 0, | ||
"mocha/no-exclusive-tests": "error", | ||
"space-before-function-paren": 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.
The space-before-function-paren
rule conflicts with standard prettier formatting which consistently doesn't add the space.
"singleQuote": true, | ||
"parser": "flow", | ||
"printWidth": 120, | ||
"trailingComma": "none", |
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.
Your eslint config was complaining about trailing commas, so I've switched it off here
* master: Align prettier settings and eslint (Atos-Research-and-Innovation#49)
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
With the previous PR #47 - I broke your linting rules. This fixes the issue by either amending the prettier config or the eslint config so that they match.
It shouldn't be an issue going forward as prettier will do its thing on commit, I've also added the linter to the travis CI so that this gets picked up properly.