Skip to content

check whether compiler lsp support compile project #7374

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

Merged
merged 7 commits into from
May 21, 2025

Conversation

chunyu3
Copy link
Contributor

@chunyu3 chunyu3 commented May 16, 2025

When leverage lsp to emit code, the lsp server need have the capacity of internalCompile. Typespec compiler support this only after version 1.0.0. So check the lsp capacity before emit code, and provide clear information to info user to upgrade their compiler.

@microsoft-github-policy-service microsoft-github-policy-service bot added the ide Issues for VS, VSCode, Monaco, etc. label May 16, 2025
Copy link
Contributor

github-actions bot commented May 16, 2025

All changed packages have been documented.

  • typespec-vscode
Show changes

typespec-vscode - fix ✏️

Check whether the compiler language server supports project compilation.

@chunyu3 chunyu3 force-pushed the lspSupportCheck branch from 2e471a1 to f9bd277 Compare May 16, 2025 07:08
@azure-sdk
Copy link
Collaborator

azure-sdk commented May 16, 2025

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@chunyu3 chunyu3 requested a review from lirenhe May 16, 2025 07:27
@chunyu3 chunyu3 force-pushed the lspSupportCheck branch from 34e5447 to 1aec717 Compare May 16, 2025 07:30
@chunyu3 chunyu3 force-pushed the lspSupportCheck branch from 1aec717 to 2d9d6a6 Compare May 16, 2025 07:31
@chunyu3 chunyu3 requested a review from RodgeFu May 19, 2025 03:45
@chunyu3 chunyu3 added this pull request to the merge queue May 21, 2025
Merged via the queue into microsoft:main with commit f577067 May 21, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ide Issues for VS, VSCode, Monaco, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants