Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(TF-19544) fix: Support reference completions with a trailing dot in object constructors #410

Merged
merged 3 commits into from
Aug 26, 2024

Conversation

ansgarm
Copy link
Member

@ansgarm ansgarm commented Aug 21, 2024

This depends on hashicorp/hcl#692

  • fix: Support reference completions after dot in object constructors
  • Bump hcl to 117baa8

completions

@ansgarm ansgarm requested a review from a team as a code owner August 21, 2024 14:04
@ansgarm ansgarm changed the base branch from main to pre-release August 21, 2024 14:05
@ansgarm ansgarm changed the title fix ref completions after dot in objects (TF-19544) fix: Support reference completions with a trailing dot in object constructors Aug 21, 2024
@ansgarm ansgarm force-pushed the fix-ref-completions-after-dot-in-objects branch from 4ea6da1 to 107393f Compare August 21, 2024 14:15
Copy link
Contributor

@jpogran jpogran left a comment

Choose a reason for hiding this comment

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

Might this go to main as it's not specific to stacks?

@ansgarm
Copy link
Member Author

ansgarm commented Aug 21, 2024

Might this go to main as it's not specific to stacks?

Good call! Yeah 👍 I'll rebase

@ansgarm ansgarm force-pushed the fix-ref-completions-after-dot-in-objects branch from 0267f45 to 8bb6cdb Compare August 22, 2024 09:29
@ansgarm ansgarm changed the base branch from pre-release to main August 22, 2024 09:29
@ansgarm ansgarm force-pushed the fix-ref-completions-after-dot-in-objects branch from 8bb6cdb to 7c543c8 Compare August 22, 2024 09:31
Previously it was required to type a first character as the hclsyntax parser wouldn't return anything for these invalid references. This requires hashicorp/hcl#692 to work
@ansgarm ansgarm force-pushed the fix-ref-completions-after-dot-in-objects branch from 7c543c8 to f4ee4cc Compare August 22, 2024 09:32
@ansgarm ansgarm requested a review from jpogran August 22, 2024 10:35
Copy link
Contributor

@jpogran jpogran left a comment

Choose a reason for hiding this comment

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

❤️

@ansgarm ansgarm force-pushed the fix-ref-completions-after-dot-in-objects branch from f4ee4cc to 9cbeb16 Compare August 26, 2024 12:51
@ansgarm ansgarm merged commit daf20a7 into main Aug 26, 2024
5 checks passed
@ansgarm ansgarm deleted the fix-ref-completions-after-dot-in-objects branch August 26, 2024 12:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants