Skip to content

fix: enforce checked cleanup and output handling#4

Merged
zzwong merged 4 commits into
mainfrom
ci/golangci-lint
Jul 13, 2026
Merged

fix: enforce checked cleanup and output handling#4
zzwong merged 4 commits into
mainfrom
ci/golangci-lint

Conversation

@zzwong

@zzwong zzwong commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • handle previously unchecked output, response-body, archive, and temporary-file cleanup errors
  • add a minimal golangci-lint v2 configuration with no broad suppressions
  • run pinned golangci-lint v2.12.2 for code changes and include it in the required CI gate
  • add a dynamic maintained lint badge and update contributor commands
  • add end-to-end synthetic tests for Green Button commands, Opower usage resources, realtime reads, summaries, and entity discovery
  • raise total statement coverage from 61.6% to 71.0% and enforce a 70% floor

Testing

  • make check
  • govulncheck ./...
  • golangci-lint config verify
  • actionlint .github/workflows/*.yml
  • total statement coverage: 71.0%

Security and privacy

  • no production fixtures or provider values changed
  • all new provider responses are synthetic
  • cleanup errors are either propagated or explicitly ignored only on best-effort cleanup paths

@zzwong zzwong merged commit bdf799a into main Jul 13, 2026
6 checks passed
@zzwong zzwong deleted the ci/golangci-lint branch July 13, 2026 08:59
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