chore: remove unnecessary useless comments (#219)

Author: sd2k

tools/alerting_client.go

@@ -82,7 +82,7 @@ func (c *alertingClient) makeRequest(ctx context.Context, path string) (*http.Re
 	}
 	if resp.StatusCode != http.StatusOK {
 		bodyBytes, _ := io.ReadAll(resp.Body)
-		_ = resp.Body.Close() //nolint:errcheck // Ignore close error in error handling path
+		_ = resp.Body.Close() //nolint:errcheck
 		return nil, fmt.Errorf("grafana API returned status code %d: %s", resp.StatusCode, string(bodyBytes))
 	}
 
@@ -95,7 +95,7 @@ func (c *alertingClient) GetRules(ctx context.Context) (*rulesResponse, error) {
 		return nil, fmt.Errorf("failed to get alert rules from Grafana API: %w", err)
 	}
 	defer func() {
-		_ = resp.Body.Close() //nolint:errcheck // Ignore close error in defer
+		_ = resp.Body.Close() //nolint:errcheck
 	}()
 
 	var rulesResponse rulesResponse

tools/asserts.go

@@ -91,7 +91,7 @@ func (c *Client) fetchAssertsData(ctx context.Context, urlPath string, method st
 		return "", fmt.Errorf("failed to execute request: %w", err)
 	}
 	defer func() {
-		_ = resp.Body.Close() //nolint:errcheck // Ignore close error in defer
+		_ = resp.Body.Close() //nolint:errcheck
 	}()
 
 	body, err := io.ReadAll(resp.Body)

tools/loki.go

@@ -114,7 +114,7 @@ func (c *Client) makeRequest(ctx context.Context, method, urlPath string, params
 		return nil, fmt.Errorf("executing request: %w", err)
 	}
 	defer func() {
-		_ = resp.Body.Close() //nolint:errcheck // Ignore close error in defer
+		_ = resp.Body.Close() //nolint:errcheck
 	}()
 
 	// Check for non-200 status code

tools/oncall.go

@@ -34,7 +34,7 @@ func getOnCallURLFromSettings(ctx context.Context, grafanaURL, grafanaAPIKey str
 		return "", fmt.Errorf("fetching settings: %w", err)
 	}
 	defer func() {
-		_ = resp.Body.Close() //nolint:errcheck // Ignore close error in defer
+		_ = resp.Body.Close() //nolint:errcheck
 	}()
 
 	if resp.StatusCode != http.StatusOK {

tools/pyroscope.go

@@ -371,7 +371,7 @@ func (c *pyroscopeClient) get(ctx context.Context, path string, params url.Value
 		return nil, fmt.Errorf("failed to send request: %w", err)
 	}
 	defer func() {
-		_ = res.Body.Close() //nolint:errcheck // Ignore close error in defer
+		_ = res.Body.Close() //nolint:errcheck
 	}()
 
 	if res.StatusCode < 200 || res.StatusCode > 299 {

tools/sift.go

@@ -450,7 +450,7 @@ func (c *siftClient) makeRequest(ctx context.Context, method, path string, body
 		return nil, fmt.Errorf("executing request: %w", err)
 	}
 	defer func() {
-		_ = response.Body.Close() //nolint:errcheck // Ignore close error in defer
+		_ = response.Body.Close() //nolint:errcheck
 	}()
 
 	// Check for non-200 status code (matching Loki client's logic)