-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
[explicit-member-accessibility] cannot be configured #422
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
Comments
We haven't released that feature to main yet. Please use the |
@bradzacher, if it's not released yet, why is it documented in master? |
We don't have a separate release branch, master is our pr target branch because it makes life easier. We release to the main tag on npm every week or two. It's a manual release so it happens when we get to it. We have CI to automatically run a canary release for every master commit. If you want the bleeding edge always without waiting, just install from the canary tag. |
Ahh cool, didn't take into account what your release cycle looked like, thanks for the explanation! |
@bradzacher Is there a list of the rules that are currently supported in master? |
@dlhart - do you mean currently supported against the Because the answer to your question is whatever is in master is what's supported in master...... For the code that's currently released as The code that's currently released as |
Any configuration arguments beyond the first ("error") generate an error on startup.
Repro
"@typescript-eslint/explicit-member-accessibility": ["error", { "accessibility": "explicit" }],
Expected Result
Should be able to configure the rule.
Actual Result
Linter exits with a fatal error before processing any files.
Additional Info
Full configuration at https://github.com/viridia/klendathu/blob/master/.eslintrc.js
Versions
@typescript-eslint/eslint-plugin
1.6.0
@typescript-eslint/parser
1.6.0
TypeScript
3.3.3333
ESLint
5.16.0
node
v10.11.0
npm
6.9.0
The text was updated successfully, but these errors were encountered: