Skip to content

Address some issues in Terraform docs#2934

Merged
cailyoung merged 2 commits into
mainfrom
cail/fix-tf-issues
Dec 4, 2025
Merged

Address some issues in Terraform docs#2934
cailyoung merged 2 commits into
mainfrom
cail/fix-tf-issues

Conversation

@cailyoung
Copy link
Copy Markdown
Contributor

@cailyoung cailyoung commented Nov 24, 2025

Resolves #2898

  • Image moved into the Frontmatter CMS
  • tab/space resolved
  • second example updated to align with HCP Terraform's name validation

@steve-fenton-octopus
Copy link
Copy Markdown
Contributor

I'm just going to throw this on staging to check something.

I'm not sure hcl is a supported language in Shiki, but I know the ruby support seems to work for hcl and ocl code examples.

@steve-fenton-octopus
Copy link
Copy Markdown
Contributor

HCL works!

HCL code formatted by Shiki

Copy link
Copy Markdown
Contributor

@steve-fenton-octopus steve-fenton-octopus left a comment

Choose a reason for hiding this comment

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

LGTM

@cailyoung
Copy link
Copy Markdown
Contributor Author

@steve-fenton-octopus I checked it locally first :P - the highlighting is a bit different between the two so figured it was worth addressing.

I'm going to push another change as requested by the issue author and then re-request review, as it alters our example a bit.

@cailyoung cailyoung merged commit 3b50eb4 into main Dec 4, 2025
4 checks passed
@cailyoung cailyoung deleted the cail/fix-tf-issues branch December 4, 2025 22: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.

Terraform environment prep doc issues

2 participants