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
standard 14 reports parsing errors for valid public static class fields #1524
Comments
It appears that eslint comments don't work as a workaround for this either: class AlgoliaIndex {
static maxRecordLength = 8000 // eslint-disable-line
} still causes this error:
|
Did you find a work-around @zeke? |
Static class features is still Stage 3 and it's ESLint policy to only support Stage 4 proposals. In the meantime, you can enable support with Also, duplicate of #1478 |
UPDATE: So according to #1601 (comment) my configuration below is not supported in Trying to add support for this Stage 3 proposal. Tried using the How do I use experimental JavaScript (ES Next) features? section, and added the following to my "standard": {
"parser": "@babel/eslint-parser"
} Running
Tried also adding the config to "standard": {
"parser": "@babel/eslint-parser",
"parserOptions": {
"babelOptions": {
"configFile": "./config/babel.config.json"
}
}
}
} |
I noticed that there's new rules being added to eslint that support this functionality. Would updating the eslint dependency in standard be enough to enable the parsing? Do we need to wait for all the private rules to be finished before it gets added into standard, or would the partial set that's already complete be enough? |
Ah, looks like the new features are in eslint@8.0.0-beta.1, so I'm guessing we'd need to wait for the next stable version to be released. |
What version of this package are you using?
14.3.4
What operating system, Node.js, and npm version?
macOS Catalina, Node.js 12.8.0, npm 6.14.3
What happened?
This is a repeat of #940 where the use of a public static class field:
Causes this error:
Repros:
What did you expect to happen?
I do not expect to see any errors.
Are you willing to submit a pull request to fix this bug?
Perhaps!?
The text was updated successfully, but these errors were encountered: