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

Allow JS manipulation into install API #4696

Closed
IlCallo opened this issue Jul 19, 2019 · 2 comments
Closed

Allow JS manipulation into install API #4696

IlCallo opened this issue Jul 19, 2019 · 2 comments

Comments

@IlCallo
Copy link
Member

IlCallo commented Jul 19, 2019

Is your feature request related to a problem? Please describe.
I'm trying to update TypeScript linting from deprecated TSLint to ESLint TypeScript plugin.
To do so, I have to update .eslintrc.js file, but there isn't an API to do so, like there is for JSON files. Such API could even not be possible, due to the dynamic nature of JS.
The process is tricky, because I should remove some entries, probably rearrange their order and add some in a specific order, which also dependes from linting options defined at install time (Prettier? Default? AirBnB?).

Describe the solution you'd like
I don't really have a solution, in this particular case probably the best idea is to integrate TypeScript tooling directly into the core and make it install at startup when we know about other inserted linting options. That's true especially because linting has not be thought as an AE (as far as I can see), so I have no way to know which linter preset I should prepare for, unless I manually try to detect it.

Describe alternatives you've considered
The most efficient thing I can do atm is to just copy/paste a prebuild .eslintrc.js file with all options I used, which is based on Prettier.

Additional context
See quasarframework/app-extension-typescript#18

@IlCallo
Copy link
Member Author

IlCallo commented Mar 27, 2020

This should be mitigated by ESLint into core updates when it lands
https://github.com/IlCallo/quasar/tree/eslint-into-core

@rstoenescu
Copy link
Member

This needs an AST engine and is outside of the scope of Quasar CLI...

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

No branches or pull requests

2 participants