Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[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

Closed
viridia opened this issue Apr 9, 2019 · 6 comments
Closed

[explicit-member-accessibility] cannot be configured #422

viridia opened this issue Apr 9, 2019 · 6 comments
Labels
package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin working as intended Issues that are closed as they are working as intended

Comments

@viridia
Copy link

viridia commented Apr 9, 2019

Any configuration arguments beyond the first ("error") generate an error on startup.

Repro

"@typescript-eslint/explicit-member-accessibility": ["error", { "accessibility": "explicit" }],

Error: /home/talin/Projects/k4/.eslintrc.js:
        Configuration for rule "@typescript-eslint/explicit-member-accessibility" is invalid:
        Value [{"accessibility":"explicit"}] should NOT have more than 0 items.

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

package version
@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
@viridia viridia added package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin triage Waiting for team members to take a look labels Apr 9, 2019
@bradzacher bradzacher added working as intended Issues that are closed as they are working as intended and removed triage Waiting for team members to take a look labels Apr 10, 2019
@bradzacher
Copy link
Member

We haven't released that feature to main yet.

Please use the canary tag on npm:
npm install @typescript-eslint/eslint-plugin@canary

@Xapphire13
Copy link

@bradzacher, if it's not released yet, why is it documented in master?

https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/explicit-member-accessibility.md

@bradzacher
Copy link
Member

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.
So far we haven't ever had changes between canary and master, so it's safe to use.

@Xapphire13
Copy link

Ahh cool, didn't take into account what your release cycle looked like, thanks for the explanation!

@dlhart
Copy link

dlhart commented Jul 9, 2019

@bradzacher Is there a list of the rules that are currently supported in master?

@bradzacher
Copy link
Member

@dlhart - do you mean currently supported against the latest tag on npm or the canary tag on npm?

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 latest - we generate git tags and release notes on every single latest release. You can use github's tag navigation to see exactly what is supported by a specific version.

The code that's currently released as canary is just whatever is in master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin working as intended Issues that are closed as they are working as intended
Projects
None yet
Development

No branches or pull requests

4 participants