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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

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

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