Skip to content
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

Making default extenders optional #182

Closed
camilleriluke opened this issue Feb 12, 2017 · 5 comments
Closed

Making default extenders optional #182

camilleriluke opened this issue Feb 12, 2017 · 5 comments

Comments

@camilleriluke
Copy link

My team and I have our own curated list of eslint configs that we use different extenders based on file patterns in our codebase. One thing that I found a bit not so configurable is the hardcoded extenders that are shipped with the plugin (source).

I understand that they could be turned off one by one but that is quite a maintenance cost, if there is another way how we could achieve this please let me know.

Proposed update

Alternative I would like to put forward a suggestion that could solve this.

Provide a flag (skip-default-extenders) that when turned on it will skip the default extenders. This flag will be false by default, so it will still respect the zero-config mantra of the plugin

@sindresorhus
Copy link
Member

Why even use XO if you want to skip all the built-in rules? Seems like just using ESLint would be better, or am I missing something?

@camilleriluke
Copy link
Author

I'm using it mostly for the glob based linting.

@marionebl
Copy link
Contributor

I feel this should not be part of xo. If there is a use case for the globbing/ignoring/overrides in xo without its rulesets the functionality could be moved to a separate module and cli tool.

@sindresorhus
Copy link
Member

I've been thinking about this for a while, and while it's a small change, it changes the scope of XO dramatically and opens the door to lots of other requests.

It also seems like ESLint will soon have support for glob-based config anyhow: eslint/eslint#8081

@camilleriluke
Copy link
Author

Thanks for your time anyway 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants