From b02747493a501c7edd7c33fa5d8c852f238c88ed Mon Sep 17 00:00:00 2001 From: Eric Phillips Date: Sun, 24 May 2026 02:39:49 -0600 Subject: [PATCH] fix linting errors --- internal/repomaint/db.go | 8 +++++++- internal/repomaint/repomaint_test.go | 1 + 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/internal/repomaint/db.go b/internal/repomaint/db.go index 15454aa..859bc92 100644 --- a/internal/repomaint/db.go +++ b/internal/repomaint/db.go @@ -25,7 +25,9 @@ func (r *RepoSync) buildMap(repo string) (map[string]string, error) { if err != nil { return nil, err } + //nolint:errcheck defer fr.Close() + //nolint:errcheck defer gzr.Close() // for entry in db @@ -63,7 +65,9 @@ func (r *RepoSync) updatablePkgs(repo string, pkgs map[string]string) ([]string, if err != nil { return nil, err } + //nolint:errcheck defer fr.Close() + //nolint:errcheck defer gzr.Close() files := []string{} @@ -128,7 +132,9 @@ func (r *RepoSync) openDb(repo string) (*tar.Reader, *os.File, *gzip.Reader, err } gr, err := gzip.NewReader(f) if err != nil { - f.Close() + if err := f.Close(); err != nil { + slog.Warn("failed to close gzip reader", "err", err) + } return nil, nil, nil, err } tr := tar.NewReader(gr) diff --git a/internal/repomaint/repomaint_test.go b/internal/repomaint/repomaint_test.go index 3a4890d..e6b57f6 100644 --- a/internal/repomaint/repomaint_test.go +++ b/internal/repomaint/repomaint_test.go @@ -62,6 +62,7 @@ func TestCachedFiles(t *testing.T) { } cachedFiles, err := rs.getCachedPkgs("core") + require.NoError(t, err) assert.Contains(t, cachedFiles, "linux") assert.Contains(t, cachedFiles, "gcc-rust") assert.Contains(t, cachedFiles, "linux-api-headers")