-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Getting the new parser into AST Explorer #125
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
Comments
FYI, fkling/astexplorer#377. |
Web demo would be cool. |
Web demos for eslint are hard. I started working on a universal eslint repl module a while ago (the idea being you'd self host it, pass it your plugin, and it would give you a UI that is AST explorer + rule checker in one that users can link to in issues). However the problem is that a lot of eslint essentially requires access to the filesystem via There are packages like eslint-web which are modified versions of eslint to work in the browser, but they lag behind and don't work as well as you might like. For us to create a web demo, it would probably be a fair chunk of work. |
@armano2 has pointed out that eslint-plugin-vue has this setup for their repo, so we can look to leveraging(/stealing) their work to make interactive docs happen: https://mysticatea.github.io/eslint-plugin-es/rules/no-json-superset.html Note that even with this codebase as a starting point - this is still a non-trivial amount of work due to the need for build tooling and such. |
I think it's not a steal :D Also, https://mysticatea.github.io/vue-eslint-demo/ is an existing playground, which supports old |
I'm sorry, I'm not sure if I can work in this weekend because of the flu. 🤒 |
Looks like the reason for the initial issue has been resolved, as AST explorer added |
A separate interactive playground is definitely a nice to have too :) |
AST Explorer has the latest typescript-eslint-parser (from the ESLint org) available for use, but we'll want to get the new parser out there.
Anyone know where to start?
Vaguely related question: Do we intend to set up a web demo somewhere?
(Might be off-topic as this is not strictly an issue in this repo-- apologies if that's the case)
The text was updated successfully, but these errors were encountered: