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

fix(graphql): Prevent the graphql server from accidentally starting with a TypeSpec file #4639

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ncaq
Copy link
Contributor

@ncaq ncaq commented Dec 13, 2024

The extension determination was not accurate and reacted to `.tsp' files.
Created a unit test case to the function that performs the extension determination and modified the actual code.

ncaq added 2 commits December 13, 2024 18:24
Reproduce a situation where GraphQL is unintentionally reacting to a TypeSpec file (`.tsp`).
Do not mistakenly assume that `.tsp` is a file containing GraphQL.
@github-actions github-actions bot added the client One or more of lsp-mode language clients label Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client One or more of lsp-mode language clients
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants