-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
feat(plugins): add plugin for xo config #621
Conversation
Awesome! This is great, the only remark I'd have is that there could be a {
"scripts": {
"test": "xo && ava"
}
} Happy to merge this! |
Adds a plugin for [xo](https://github.com/xojs/xo#config). Essentially its an opinionated wrapper around eslint with a bunch of bundled rules and plugins, and the config object is just the eslint config object with some extra options. So to keep it consistent with the eslint plugin and avoid duplication I've just used the eslint helper to parse out any additional eslint plugins that have been added in the xo config.
@webpro done, thanks for the suggestion |
Sorry I forgot to tell more about how binaries work, I've just added it to prevent some ping-pong. Hope you don't mind :) |
🚀 This pull request is included in v5.13.0. See Release 5.13.0 for release notes. Using Knip in a commercial project? Please consider sponsoring me. |
Thanks again @bbeesley! |
Adds a plugin for xo. Essentially its an opinionated wrapper around eslint with a bunch of bundled rules and plugins, and the config object is just the eslint config object with some extra options. So to keep it consistent with the eslint plugin and avoid duplication I've just used the eslint helper to parse out any additional eslint plugins that have been added in the xo config.