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: unify and enforce import style to no extensions #3022
chore: unify and enforce import style to no extensions #3022
Conversation
Deploy preview for algolia-instantsearch ready! Built with commit 20ad063 https://deploy-preview-3022--algolia-instantsearch.netlify.com |
@@ -3,6 +3,7 @@ module.exports = { | |||
rules: { | |||
"no-param-reassign": 0, | |||
"import/no-extraneous-dependencies": 0, | |||
"react/no-string-refs": 1 | |||
"react/no-string-refs": 1, | |||
"import/extensions": [2, "always", {js: "never"}] |
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.
What about having this rule part of eslint-config-algolia?
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.
It's a first step :) But sure it's gonna end up there if we're happy about it.
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.
algolia/eslint-config-algolia#92 we'll remove the rule when it's merged and the config updated.
…clearRefinements-import-no-extension
* refactor(clearAll): rename clear all to clear refinements * feat(clear-refinements): update markup * feat(panel): upate dev-novel and misc. fixes * chore(clear-refinements): remove panel * chore: peer review * chore: unify and enforce import style to no extensions (#3022)
Following up on @samouss comment, this PR activates import/extensions and configures to not accepts extensions for JS files.