refactor: better all-repository ruleset naming

bruzina · bruzina · commit c8a5621d7a8e · 2025-07-04T17:58:02.000+02:00
diff --git a/terraform/main.tf b/terraform/main.tf
@@ -37,10 +37,10 @@ resource "github_repository" "repo" {
   delete_branch_on_merge = try(each.value.delete_branch_on_merge, local.all_repositories.delete_branch_on_merge, null)
 }
 
-resource "github_repository_ruleset" "this" {
+resource "github_repository_ruleset" "all_repositories" {
   for_each = {
     for repository_ruleset in local.all_repositories_rulesets :
-    "${repository_ruleset.repository.name}-${repository_ruleset.ruleset.name}" => repository_ruleset
+    format("%s/%s", "${repository_ruleset.repository.name}", replace(lower("${repository_ruleset.ruleset.name}"), " ", "_")) => repository_ruleset
   }
 
   # Metadata

Original file line number	Diff line number	Diff line change
`@@ -37,10 +37,10 @@ resource "github_repository" "repo" {`
`37`	`37`	`delete_branch_on_merge = try(each.value.delete_branch_on_merge, local.all_repositories.delete_branch_on_merge, null)`
`38`	`38`	`}`
`39`	`39`
`40`		`-resource "github_repository_ruleset" "this" {`
	`40`	`+resource "github_repository_ruleset" "all_repositories" {`
`41`	`41`	`for_each = {`
`42`	`42`	`for repository_ruleset in local.all_repositories_rulesets :`
`43`		`- "${repository_ruleset.repository.name}-${repository_ruleset.ruleset.name}" => repository_ruleset`
	`43`	`+ format("%s/%s", "${repository_ruleset.repository.name}", replace(lower("${repository_ruleset.ruleset.name}"), " ", "_")) => repository_ruleset`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`# Metadata`