Skip to content

Conversation

anubhav-goel
Copy link
Contributor

@anubhav-goel anubhav-goel commented Jul 1, 2025

Description

This provides completion, hover, and diagnostics for Terraform Search files. Adds new language id - terraform-search

Related PRs:

Demo

TODO

  • Release terraform-ls and use respective version.

@anubhav-goel anubhav-goel force-pushed the feature/tfsearch-TF-26847 branch from 60d3c5b to c7a3961 Compare August 6, 2025 05:46
@anubhav-goel anubhav-goel changed the title feat(tfsearch): TF-26847 TF-27259: Added: support for .tfquery.hcl file Support for Terraform Search (.tfquery.hcl) files Aug 6, 2025
@anubhav-goel anubhav-goel marked this pull request as ready for review August 11, 2025 05:58
@anubhav-goel anubhav-goel requested a review from a team as a code owner August 11, 2025 05:58
Copy link
Contributor

@jpogran-hashi jpogran-hashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall everything needed is implemented 👍

As an aside, you don't have to duplicate the Terraform Stacks svg. The new svg exists for these reasons, so you coudl rename the file and the references and use it for query as well. Saves bundle size slightly

@anubhav-goel
Copy link
Contributor Author

Overall everything needed is implemented 👍

As an aside, you don't have to duplicate the Terraform Stacks svg. The new svg exists for these reasons, so you coudl rename the file and the references and use it for query as well. Saves bundle size slightly

Sure, done, thanks!! Renamed filename to terraform_feature.svg

Copy link
Contributor

@sunnyhashi sunnyhashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anubhav-goel anubhav-goel merged commit 3d0eac4 into main Aug 28, 2025
22 of 23 checks passed
@anubhav-goel anubhav-goel deleted the feature/tfsearch-TF-26847 branch August 28, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants