johno/beads
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(lint): check error return from provider.Close() (#1250)

Browse Source 
Two lint errors introduced in recent commits:
- cmd/bd/doctor/git.go:896 - unchecked provider.Close()
- cmd/bd/orphans.go:120 - unchecked provider.Close()

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Steve Yegge
2026-01-21 21:11:33 -08:00
committed by GitHub
parent 2cc96197c0
commit abd3feb761
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cmd/bd/doctor/git.go
View File

@@ -893,7 +893,7 @@ func FindOrphanedIssuesFromPath(path string) ([]OrphanIssue, error) {
if err != nil { if err != nil {
return []OrphanIssue{}, nil return []OrphanIssue{}, nil
} }
defer provider.Close() defer func() { _ = provider.Close() }()
return FindOrphanedIssues(path, provider) return FindOrphanedIssues(path, provider)
} }

2
cmd/bd/orphans.go
View File

@@ -117,7 +117,7 @@ func getIssueProvider() (types.IssueProvider, func(), error) {
if err != nil { if err != nil {
return nil, nil, fmt.Errorf("failed to open database at %s: %w", dbPath, err) return nil, nil, fmt.Errorf("failed to open database at %s: %w", dbPath, err)
} }
return provider, func() { provider.Close() }, nil return provider, func() { _ = provider.Close() }, nil
} }
// Use the global store (already opened by PersistentPreRun) // Use the global store (already opened by PersistentPreRun)