🚨 Silence linter warnings I don't care about

This commit is contained in:
Dan Jones 2025-07-16 11:13:53 -05:00
commit 72295c53cf
3 changed files with 14 additions and 2 deletions

View file

@ -172,6 +172,7 @@ func (m *Merger) getRecipes(ctx context.Context, items []Item) (err error) {
return return
} }
//nolint:errcheck // I don't care
defer rows.Close() defer rows.Close()
rm := make(map[int32]map[int32][]int32) rm := make(map[int32]map[int32][]int32)
var found bool var found bool
@ -214,11 +215,11 @@ func addRecs(recs *[][]int32, rec []int32) {
func recSorter(rec []int32) func([]int32) bool { func recSorter(rec []int32) func([]int32) bool {
return func(v []int32) bool { return func(v []int32) bool {
slices.Sort(v) slices.Sort(v)
if !(len(v) == len(rec)) { if len(v) != len(rec) {
return false return false
} }
for idx, i := range v { for idx, i := range v {
if !(i == rec[idx]) { if i != rec[idx] {
return false return false
} }
} }
@ -256,6 +257,7 @@ func (m *Merger) insertGames(ctx context.Context) (mp map[int64]Game, err error)
if err != nil { if err != nil {
return return
} }
//nolint:errcheck // I don't care
defer stmt.Close() defer stmt.Close()
for _, g := range m.games { for _, g := range m.games {
res, err = stmt.ExecContext(ctx, g) res, err = stmt.ExecContext(ctx, g)
@ -289,11 +291,13 @@ func (m *Merger) insertItems(ctx context.Context, gameID int64, items []Item) (i
if err != nil { if err != nil {
return return
} }
//nolint:errcheck // I don't care
defer stmt.Close() defer stmt.Close()
fetchIDStmt, err = m.db.PreparexContext(ctx, `SELECT id FROM items WHERE name = ?`) fetchIDStmt, err = m.db.PreparexContext(ctx, `SELECT id FROM items WHERE name = ?`)
if err != nil { if err != nil {
return return
} }
//nolint:errcheck // I don't care
defer fetchIDStmt.Close() defer fetchIDStmt.Close()
relStmt, err = m.db.PreparexContext(ctx, ` relStmt, err = m.db.PreparexContext(ctx, `
INSERT INTO games_items INSERT INTO games_items
@ -303,6 +307,7 @@ func (m *Merger) insertItems(ctx context.Context, gameID int64, items []Item) (i
if err != nil { if err != nil {
return return
} }
//nolint:errcheck // I don't care
defer relStmt.Close() defer relStmt.Close()
for _, it := range items { for _, it := range items {
_, err = stmt.ExecContext(ctx, it) _, err = stmt.ExecContext(ctx, it)
@ -334,6 +339,7 @@ func (m *Merger) insertRecipes(ctx context.Context, gameID, itemID int64, recipe
if err != nil { if err != nil {
return return
} }
//nolint:errcheck // I don't care
defer insertStmt.Close() defer insertStmt.Close()
insertRecItemStmt, err = m.db.PreparexContext(ctx, ` insertRecItemStmt, err = m.db.PreparexContext(ctx, `
INSERT INTO recipes_items INSERT INTO recipes_items
@ -343,6 +349,7 @@ func (m *Merger) insertRecipes(ctx context.Context, gameID, itemID int64, recipe
if err != nil { if err != nil {
return return
} }
//nolint:errcheck // I don't care
defer insertRecItemStmt.Close() defer insertRecItemStmt.Close()
selStmt, err = m.db.PreparexContext(ctx, ` selStmt, err = m.db.PreparexContext(ctx, `
SELECT item_id FROM games_items SELECT item_id FROM games_items
@ -351,6 +358,7 @@ func (m *Merger) insertRecipes(ctx context.Context, gameID, itemID int64, recipe
if err != nil { if err != nil {
return return
} }
//nolint:errcheck // I don't care
defer selStmt.Close() defer selStmt.Close()
for _, recipe := range recipes { for _, recipe := range recipes {
var recipeID int64 var recipeID int64

View file

@ -14,6 +14,7 @@ func NewMerger(args []string) (*Merger, error) {
} }
fs := make([]*os.File, len(args)) fs := make([]*os.File, len(args))
for idx, pth := range args { for idx, pth := range args {
//nolint:gosec // Yes, of course
fs[idx], err = os.Open(pth) fs[idx], err = os.Open(pth)
if err != nil { if err != nil {
return nil, err return nil, err

View file

@ -14,12 +14,15 @@ func (m *Merger) Write(ctx context.Context, g Game) (out string, err error) {
out = base + ".ic" out = base + ".ic"
var f *os.File var f *os.File
//nolint:gosec // Yes, of course I'm getting the file from user input
if f, err = os.Create(out); err != nil { if f, err = os.Create(out); err != nil {
return return
} }
//nolint:errcheck // I don't care
defer f.Close() defer f.Close()
gz := gzip.NewWriter(f) gz := gzip.NewWriter(f)
//nolint:errcheck // I don't care
defer gz.Close() defer gz.Close()
gz.Name = base gz.Name = base
gz.ModTime = time.Unix(g.Updated/1000, g.Updated%1000) gz.ModTime = time.Unix(g.Updated/1000, g.Updated%1000)