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

parse without eslint #92

Open
benmccann opened this issue Sep 12, 2024 · 3 comments
Open

parse without eslint #92

benmccann opened this issue Sep 12, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request priority:high

Comments

@benmccann
Copy link

I was wondering if there might be another way to parse such as using zimmerframe because eslint is a very heavy dependency

https://npmgraph.js.org/?q=sveltedoc-parser

@benmccann
Copy link
Author

It looks like it's being used for comments, but I'm not sure it's necessary. This method seems to be able to handle comments without using eslint:

https://github.com/xeho91/svelte-ast-print/blob/5fa160ad48db333d95e695b5d36e3a4e7fdd28c0/tests/nodes/script.test.ts#L84

@Sidnioulz
Copy link

ESLint 8, which this package uses, has recently been deprecated. @alexprey, do you by any chance have plans to update to ESLint 9 or replace the parser?

@benbender
Copy link

Given the fact that @alexprey (sadly) wasn't active on github for at least over a year, this might be a dead end and the lib should be considered unmaintained…

I hope he is doing fine and it's just a matter of priorities, but it might be worth discussing to fork the lib to get it up to date with dependencies and requirements by the projects that rely on it…

@KatChaotic KatChaotic self-assigned this Oct 25, 2024
@KatChaotic KatChaotic added the enhancement New feature or request label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority:high
Projects
None yet
Development

No branches or pull requests

4 participants