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

Enable basic CMake language services #4204

Merged
merged 12 commits into from
Jan 10, 2025

Conversation

gcampbell-msft
Copy link
Collaborator

This PR accomplishes the following:

  • Removes the dependency on the third party twxs.cmake extension. Thanks @twxs for your work in that extension!
  • Add support into our extension for the following:
    • Quick Info for built-in modules, variables, properties, and functions.
    • Completions for built-in modules, variables, properties, and functions.

This does this by including JSON files in our repository that house information for language services and we load these on activation of the extension. This enhances the performance of providing completions and quick info information.

The main implementation points in this PR:

  • Bundling the language services json assets into the vsix (gulpfile.js, etc.)
  • Integrating with the VS Code Language Service API to implement quick hover and completions (src/languageServices)

Fixes #3559. Fixes #534. Fixes #3560

@gcampbell-msft gcampbell-msft merged commit d3f8f05 into main Jan 10, 2025
4 checks passed
@gcampbell-msft gcampbell-msft deleted the dev/gcampbell/ColorizationAndQuickInfo branch January 10, 2025 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants