Skip to content

fix: use vue-eslint-parser directly#1277

Merged
susnux merged 1 commit intomainfrom
chore/use-vue-parser
Jan 20, 2026
Merged

fix: use vue-eslint-parser directly#1277
susnux merged 1 commit intomainfrom
chore/use-vue-parser

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jan 20, 2026

Resolves a breaking change with eslint-plugin-vue.

@susnux susnux requested review from Antreesy and ShGKme January 20, 2026 09:51
@susnux susnux added bug Something isn't working 3. to review labels Jan 20, 2026
@ShGKme
Copy link
Contributor

ShGKme commented Jan 20, 2026

Does it copy-paste or reimplement functions from vue-eslint-plugin?

@susnux
Copy link
Contributor Author

susnux commented Jan 20, 2026

Does it copy-paste or reimplement functions from vue-eslint-plugin?

Partly copy past but with adjusted types.
Basically all it does is emit a warning if vue-eslint-parser is not available and otherwise use the template visitor from vue-eslint-parser

Resolves a breaking change with eslint-plugin-vue.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the chore/use-vue-parser branch from e834b18 to 514b897 Compare January 20, 2026 17:37
@susnux susnux merged commit 67d24d4 into main Jan 20, 2026
7 checks passed
@susnux susnux deleted the chore/use-vue-parser branch January 20, 2026 20:01
susnux added a commit that referenced this pull request Jan 28, 2026
changes in this version:
* fix(nextcloud-plugin): add more removed API in Nextcloud 33 [\#1269](#1269) \([susnux](https://github.com/susnux)\)
* fix: use vue-eslint-parser directly [\#1277](#1277) \([susnux](https://github.com/susnux)\)

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments