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

report errors on first character for *.vue/*.svelte code, due graphql-tag-pluck limitation we can't know right location #1346

Merged
merged 1 commit into from
Dec 27, 2022

Conversation

dimaMachina
Copy link
Collaborator

…phql-tag-pluck limitation we can't know right location
@changeset-bot
Copy link

changeset-bot bot commented Dec 27, 2022

🦋 Changeset detected

Latest commit: 9862aaf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@graphql-eslint/eslint-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@theguild-bot
Copy link
Collaborator

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
@graphql-eslint/eslint-plugin 3.14.2-alpha-20221227181549-bdccde8 npm ↗︎ unpkg ↗︎

@dimaMachina dimaMachina merged commit f031f56 into master Dec 27, 2022
@dimaMachina dimaMachina deleted the fix-vue-svelte branch December 27, 2022 18:25
@jaydenseric
Copy link

This change seems to have introduced this bug: #1348 (because according to the release descriptions nothing else changed in @graphql-eslint/eslint-plugin v3.14.2).

@bmulholland
Copy link
Contributor

Just found this issue after digging into why I wasn't seeing lint errors I was expecting in my editor.

What's the limitation here? I don't see a link to anything in graphql-tag-pluck. Is there an issue filed over there about it? Is there any hope of this being fixed? Why or why not? Anything I can do to help?

@bmulholland
Copy link
Contributor

bmulholland commented Jan 8, 2024

There's 9 total issues on that project, all closed, and none seem related (though maybe I'm missing something). I'm happy to file an issue over there to request this limitation be lifted, but I don't know anything about what it is. @dimaMachina I know it's been a year, do you recall?

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

Successfully merging this pull request may close these issues.

Support extracting GraphQL documents from vue/svelte files
4 participants