-
Notifications
You must be signed in to change notification settings - Fork 5
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
Write ESLint RFC about solving CLI needs for neostandard
, xo
etc
#33
Comments
neostandard
, xo
etc needsneostandard
, xo
etc
Posted here: eslint/eslint#18800 Closing as done for now. |
Keeping open to track progress |
Feedback on eslint/eslint#18800 (comment) etc appreciated 🙏 |
Idea for a neostandard cli: Rather than the old model used for standard, where a a parallel plugin ecosystem was required for editors and whatnot, the neostandard cli would just set up the underlying tools for neostandard use (generate eslint config, and any other eventually bundled tools like dprint or boom) and then run a check that they remain configure correctly, perhaps with a strict and loose option that allows and prohibits customization. This check could be run along side linting as desired. |
@bcomnes Building on that I was pondering whether something like One could then possibly also support eg. And when/if (An |
It could also facilitate a multi-tool future if that is ever forced (eslint + dprint for example). |
Handling current flat configurations extended with neostandard (last) with the CLI can also help potential users to answer the question: "What will be the impact of migrating a code base to neostandard?" with no configuration fuzz. |
I keep trying to find ways to express my ideas in a way that conveys it in an understandable way: eslint/eslint#18800 (comment) |
@wesleytodd I give up on trying to get through, I’m apparently not able to express it in a way that makes sense, or I focused on the wrong aspect initially and that’s what got stuck in people’s mind. Feel free to give it a try. I wonder if it would be easier to try to make PR:s to eg the VSCode ESLint extension that introduces a separate tool-specific configuration lookup. It would take more effort and be less flexible than doing it as an extension to the ESLint configuration system, but 🤷 I will however not be able to spend any time on this for the rest of the year – I have spent too much time on unpaid projects and all I get in return is massive pushback when I try to help out, and that takes some time to recover from that and I need to spend all of that time on projects that actually helps me pay my rent. |
Another option would be to fork That way it would work automatically in eg the VSCode extension I believe |
I barely have enough time for this project let alone tracking an overactive upstream project like eslint. Let's just ship our own helper cli to help with config setup. |
Well do something like what @bcomnes suggested, when someone has time for it, but I’m going to close this one as I rather put this whole thing behind me |
Replaced by: #205 |
standard-engine
solved two problems:It also came with some drawbacks:
What I would want to do is to:
Find an official solution together with the ESLint project, where we can officially bundle ESLint with our config and have it be exposed as a standalone non-configurable CLI.
That could enable editors etc to pick the bundled config without any custom integrations (I'm not very keen on going the route of
standard
and solving #4 by building new editor integrations 🙂) This could probably happen byeslint
itself gaining the ability to pick up such a bundled configuration even when launched independently rather than through the standalone CLI (neostandard
,xo
etc)To get this process started, an RFC to the ESLint project should be written.
Originally posted by @voxpelli in #16 (comment)
Also discussed in: xojs/xo#702 (comment)
The text was updated successfully, but these errors were encountered: