segraef
commented
4 months ago - [x] ID: TFFR1 - Category: Composition - Cross-Referencing Modules
- [x] ID: TFFR2 - Category: Outputs - Additional Terraform Outputs
- [x] ID: TFNFR1 - Category: Documentation - Descriptions
- [x] ID: TFNFR2 - Category: Documentation - Module Documentation Generation
- [x] ID: TFNFR3 - Category: Contribution/Support - GitHub Repo Branch Protection
- [x] ID: TFNFR4 - Category: Composition - Code Styling - lower snake_casing
- [x] ID: TFNFR5 - Category: Testing - Test Tooling
- [x] ID: TFNFR6 - Category: Code Style - The Order of resource and data in the Same File
- [ ] ID: TFNFR7 - Category: Code Style - The Use of count and for_each
- [x] ID: TFNFR8 - Category: Code Style - Orders Within resource and data Blocks
- [x] ID: TFNFR9 - Category: Code Style - Order within a module block
- [ ] ID: TFNFR10 - Category: Code Style - Values in ignore_changes passed to provider, depends_on, lifecycle blocks are not allowed to use double quotations
- [x] ID: TFNFR11 - Category: Code Style - null comparison as creation toogle
- [x] ID: TFNFR12 - Category: Code Style - Optional nested object argument should use dynamic
- [x] ID: TFNFR13 - Category: Code Style - Use coalesce or try when setting default values for nullable expressions
- [x] ID: TFFR14 - Category: Inputs - No enabled or module_depends_on variable
- [x] ID: TFNFR15 - Category: Code Style - Order to define variable
- [x] ID: TFNFR16 - Category: Code Style - Name of a variable MUST follow rules
- [x] ID: TFNFR17 - Category: Code Style - Every variable MUST come with a description
- [x] ID: TFNFR18 - Category: Code Style - Every variable MUST have an appropriate type
- [x] ID: TFNFR19 - Category: Code Style - variable containing confidential data should be declared as sensitive = true
- [x] ID: TFNFR20 - Category: Code Style - Declare nullable = false when it’s possible
- [x] ID: TFNFR21 - Category: Code Style - MUST NOT declare nullable = true
- [x] ID: TFNFR22 - Category: Code Style - MUST NOT declare sensitive = false
- [x] ID: TFNFR23 - Category: Code Style - variable with sensitive = true MUST NOT have default value
- [x] ID: TFNFR24 - Category: Code Style - Deal with deprecated variable
- [x] ID: TFNFR25 - Category: Code Style - All verified modules MUST have terraform.tf file and required_version MUST be set
- [x] ID: TFNFR26 - Category: Code Style - Providers MUST be declared in the required_providers block in terraform.tf and MUST have a constraint on minimum and maximum major version
- [x] ID: TFNFR27 - Category: Code Style - Declaration of a provider in the module
- [x] ID: TFNFR28 - Category: Code Style - output MUST be arranged alphabetically
- [x] ID: TFNFR29 - Category: Code Style - output contains confidential data should declare sensitive = true
- [x] ID: TFNFR30 - Category: Code Style - Dealing with Deprecated outputs
- [x] ID: TFNFR31 - Category: Code Style - locals.tf MUST contain only locals blocks
- [x] ID: TFNFR32 - Category: Code Style - local should be arranged alphabetically
- [x] ID: TFNFR33 - Category: Code Style - local should use types as precise as possible
- [x] ID: TFNFR34 - Category: Code Style - Feature toggle MUST be used to ensure forward compatibility of versions and avoid unexpected changes caused by upgrades
- [x] ID: TFNFR35 - Category: Code Style - Changes that might be breaking change MUST be reviewed with caution
- [x] ID: TFNFR36 - Category: Code Style - Example code MUST set prevent_deletion_if_contains_resources to false in provider block
- [x] ID: TFNFR37 - Category: Code Style - Module owner MAY use tools like newres
Update docs after tflint fix