Addressing comment 'Please don't use single char variables, it degrades readability of the code a lot'

Author: sprioriello

github/util_security_configuration.go

@@ -9,53 +9,53 @@ func flattenDependencyGraphAutosubmitActionOptions(options *github.DependencyGra
 	if options == nil {
 		return []any{}
 	}
-	m := make(map[string]any)
+	autosubmitOpts := make(map[string]any)
 	if options.LabeledRunners != nil {
-		m["labeled_runners"] = options.GetLabeledRunners()
+		autosubmitOpts["labeled_runners"] = options.GetLabeledRunners()
 	}
-	return []any{m}
+	return []any{autosubmitOpts}
 }
 
 // flattenCodeScanningDefaultSetupOptions converts CodeScanningDefaultSetupOptions to a Terraform-compatible format
 func flattenCodeScanningDefaultSetupOptions(options *github.CodeScanningDefaultSetupOptions) []any {
 	if options == nil {
 		return []any{}
 	}
-	m := make(map[string]any)
-	m["runner_type"] = options.RunnerType
+	setupOpts := make(map[string]any)
+	setupOpts["runner_type"] = options.RunnerType
 	if options.RunnerLabel != nil {
-		m["runner_label"] = options.GetRunnerLabel()
+		setupOpts["runner_label"] = options.GetRunnerLabel()
 	}
-	return []any{m}
+	return []any{setupOpts}
 }
 
 // flattenCodeScanningOptions converts CodeScanningOptions to a Terraform-compatible format
 func flattenCodeScanningOptions(options *github.CodeScanningOptions) []any {
 	if options == nil {
 		return []any{}
 	}
-	m := make(map[string]any)
+	scanOpts := make(map[string]any)
 	if options.AllowAdvanced != nil {
-		m["allow_advanced"] = options.GetAllowAdvanced()
+		scanOpts["allow_advanced"] = options.GetAllowAdvanced()
 	}
-	return []any{m}
+	return []any{scanOpts}
 }
 
 // flattenSecretScanningDelegatedBypassOptions converts SecretScanningDelegatedBypassOptions to a Terraform-compatible format
 func flattenSecretScanningDelegatedBypassOptions(options *github.SecretScanningDelegatedBypassOptions) []any {
 	if options == nil {
 		return []any{}
 	}
-	m := make(map[string]any)
+	bypassOpts := make(map[string]any)
 	if options.Reviewers != nil {
 		reviewers := make([]any, 0, len(options.Reviewers))
-		for _, r := range options.Reviewers {
-			rM := make(map[string]any)
-			rM["reviewer_id"] = r.ReviewerID
-			rM["reviewer_type"] = r.ReviewerType
-			reviewers = append(reviewers, rM)
+		for _, reviewer := range options.Reviewers {
+			reviewerMap := make(map[string]any)
+			reviewerMap["reviewer_id"] = reviewer.ReviewerID
+			reviewerMap["reviewer_type"] = reviewer.ReviewerType
+			reviewers = append(reviewers, reviewerMap)
 		}
-		m["reviewers"] = reviewers
+		bypassOpts["reviewers"] = reviewers
 	}
-	return []any{m}
+	return []any{bypassOpts}
 }