Skip to content

Fix current errcheck lint failures#110

Open
marcus wants to merge 1 commit intomainfrom
lint-fix/errcheck-current
Open

Fix current errcheck lint failures#110
marcus wants to merge 1 commit intomainfrom
lint-fix/errcheck-current

Conversation

@marcus
Copy link
Copy Markdown
Owner

@marcus marcus commented Apr 5, 2026

Summary

  • fix the currently reproducible errcheck failures in command code by handling best-effort config/database writes explicitly
  • clean up remaining errcheck findings surfaced during the same lint pass across adjacent tests and sync helpers
  • add shared test helpers for common must-succeed setup operations to keep the fixes narrow and readable

Verification

  • golangci-lint run ./...
  • go test ./cmd -run 'Start|Unstart|Ws'
  • go test ./... (still fails on the pre-existing unrelated internal/sync test TestColumnNameInjection_DroppedSilently)

Automated by nightshift

Nightshift-Task: lint-fix
Nightshift-Ref: https://github.com/marcus/nightshift
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant