Skip to content

refactor: use and enforce go fix#848

Merged
phm07 merged 2 commits intomainfrom
go-fix
May 8, 2026
Merged

refactor: use and enforce go fix#848
phm07 merged 2 commits intomainfrom
go-fix

Conversation

@phm07
Copy link
Copy Markdown
Contributor

@phm07 phm07 commented May 8, 2026

For a more detailed explanation of go fix, see https://go.dev/blog/gofix
Note: This replaces all uses of interface{} in our public API with any. This should not be a problem, since they are equivalent as per the Go docs: https://pkg.go.dev/builtin#any
Since we currently require Go 1.25 in go.mod and any has been available since Go 1.18, this does not force a Go upgrade for consumers.

@phm07 phm07 requested a review from a team as a code owner May 8, 2026 09:38
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

❌ Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.39%. Comparing base (b5d4aa6) to head (acd2b4e).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
hcloud/rdns.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #848   +/-   ##
=======================================
  Coverage   78.39%   78.39%           
=======================================
  Files          60       60           
  Lines        4355     4355           
=======================================
  Hits         3414     3414           
  Misses        637      637           
  Partials      304      304           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@phm07 phm07 merged commit 5bb7ba6 into main May 8, 2026
4 checks passed
@phm07 phm07 deleted the go-fix branch May 8, 2026 11:03
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.

2 participants