Skip to content

docs: add missing output/variable descriptions in network and ha-hot-hot modules#44

Open
dmchaledev wants to merge 1 commit into
mainfrom
claude/blissful-pascal-3l82gm
Open

docs: add missing output/variable descriptions in network and ha-hot-hot modules#44
dmchaledev wants to merge 1 commit into
mainfrom
claude/blissful-pascal-3l82gm

Conversation

@dmchaledev

Copy link
Copy Markdown
Contributor

Summary

11 output blocks and 3 variable blocks were missing description attributes across four modules. This caused blank entries in terraform-docs autogenerated docs, empty tooltips in the Terraform Registry UI, and no IDE hover-docs for those fields.

Files changed:

File Fields fixed
modules/ha-hot-hot/aws/outputs.tf alb_arn, ami_id, security_group_ids
modules/ha-hot-hot/azure/outputs.tf load_balancer_id, vm_private_ips
modules/network/aws/outputs.tf vpc_cidr, availability_zones, internet_gateway_id
modules/network/aws/variables.tf flow_logs_retention_days, tags
modules/network/azure/outputs.tf vnet_id, vnet_name, private_dns_zone_name
modules/network/azure/variables.tf tags

No logic changes — descriptions only. Zero blast radius.

Checklist

  • terraform validate — no logic changed, descriptions are pure metadata
  • tflint — description additions are not linted
  • No new checkov suppressions needed
  • No new variables, outputs, or resources added

🤖 Generated with Claude Code

https://claude.ai/code/session_01VKL2PeZmRiouMPDr8CHFiz


Generated by Claude Code

…d ha-hot-hot modules

11 output blocks and 3 variable blocks across modules/network/aws,
modules/network/azure, modules/ha-hot-hot/aws, and modules/ha-hot-hot/azure
were missing description attributes, breaking terraform-docs autogeneration
and leaving the Terraform Registry UI and IDE hover-docs blank for those
fields.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VKL2PeZmRiouMPDr8CHFiz
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.

2 participants