Fix typo in variable name nat_rule_name_shortned - #212
Conversation
📝 WalkthroughWalkthroughThis pull request corrects a spelling inconsistency in variable field names across two Terraform modules. The variable attribute 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@Dilhasha The PR is now ready for re-review. Please take a look when you have a moment. |
nat_rule_name_shortned
Purpose
Corrects a typo in the variable name
nat_rule_name_shortnedused in the Azure firewall modules.Resolves #157
Goals
Ensure correct spelling (
nat_rule_name_shortened) of the variable in variable definitions and its resource implementations.Approach
Updated references to the variable
nat_rule_name_shortnedtonat_rule_name_shortenedin the following files:modules/azurerm/Firewall-LB-DNAT-Rule-Collection/variables.tfmodules/azurerm/Firewall-LB-DNAT-Rule-Collection/firewall_dnat_rules.tfmodules/azurerm/Firewall-Multiple-PIP/variables.tfmodules/azurerm/Firewall-Multiple-PIP/nat_rules.tfRan
terraform fmt -recursiveto ensure standard HCL formatting is preserved.User stories
N/A
Release note
Corrected the typo in variable name
nat_rule_name_shortnedtonat_rule_name_shortenedin azurerm Firewall modules.Documentation
N/A - This is a bug fix for an internal variable naming typo.
Training
N/A
Certification
N/A - This change has no impact on certification exams.
Marketing
N/A
Automation tests
N/A
Validated configurations successfully via local
terraform initandterraform validateon the affected modules.Security checks
Samples
N/A
Related PRs
N/A
Migrations (if applicable)
Users of the
Firewall-LB-DNAT-Rule-CollectionandFirewall-Multiple-PIPmodules who define thenat_rule_name_shortnedattribute in theirdynamic_nat_rulesconfiguration block will need to update the key name tonat_rule_name_shortenedwhen upgrading.Test environment
Learning
N/A