fix: remove unnecessary resp nil guards in runner group Read/Update

The response is always non-nil on the success path — if the API call
errors we return early, and even the 304 Not Modified path returns
a valid response object. The fallback to the stored ETag was unreachable.

Author: austenstone

github/resource_github_actions_runner_group.go

@@ -265,10 +265,7 @@ func resourceGithubActionsRunnerGroupRead(ctx context.Context, d *schema.Resourc
 		return diag.FromErr(err)
 	}
 
-	runnerGroupEtag := normalizeEtag(d.Get("etag").(string))
-	if resp != nil {
-		runnerGroupEtag = normalizeEtag(resp.Header.Get("ETag"))
-	}
+	runnerGroupEtag := normalizeEtag(resp.Header.Get("ETag"))
 
 	runnerGroupNetworking, _, err := getRunnerGroupNetworking(client, ctx, fmt.Sprintf("orgs/%s/actions/runner-groups/%d", orgName, runnerGroupID))
 	if err != nil {
@@ -390,10 +387,7 @@ func resourceGithubActionsRunnerGroupUpdate(ctx context.Context, d *schema.Resou
 		return diag.FromErr(err)
 	}
 
-	runnerGroupEtag := normalizeEtag(d.Get("etag").(string))
-	if resp != nil {
-		runnerGroupEtag = normalizeEtag(resp.Header.Get("ETag"))
-	}
+	runnerGroupEtag := normalizeEtag(resp.Header.Get("ETag"))
 
 	if err := setGithubActionsRunnerGroupState(d, runnerGroup, runnerGroupEtag, selectedRepositoryIDs); err != nil {
 		return diag.FromErr(err)

github/resource_github_enterprise_actions_runner_group.go

@@ -251,10 +251,7 @@ func resourceGithubActionsEnterpriseRunnerGroupRead(ctx context.Context, d *sche
 		return diag.FromErr(err)
 	}
 
-	runnerGroupEtag := normalizeEtag(d.Get("etag").(string))
-	if resp != nil {
-		runnerGroupEtag = normalizeEtag(resp.Header.Get("ETag"))
-	}
+	runnerGroupEtag := normalizeEtag(resp.Header.Get("ETag"))
 
 	runnerGroupNetworking, _, err := getRunnerGroupNetworking(client, ctx, fmt.Sprintf("enterprises/%s/actions/runner-groups/%d", enterpriseSlug, runnerGroupID))
 	if err != nil {
@@ -373,10 +370,7 @@ func resourceGithubActionsEnterpriseRunnerGroupUpdate(ctx context.Context, d *sc
 		return diag.FromErr(err)
 	}
 
-	runnerGroupEtag := normalizeEtag(d.Get("etag").(string))
-	if resp != nil {
-		runnerGroupEtag = normalizeEtag(resp.Header.Get("ETag"))
-	}
+	runnerGroupEtag := normalizeEtag(resp.Header.Get("ETag"))
 
 	if err := setGithubActionsEnterpriseRunnerGroupState(d, runnerGroup, runnerGroupEtag, enterpriseSlug, selectedOrganizationIDs); err != nil {
 		return diag.FromErr(err)