Skip to content

Fix lint issues for golangci-lint v2.10.1#897

Closed
asayyah wants to merge 1 commit intomasterfrom
lint-config-update
Closed

Fix lint issues for golangci-lint v2.10.1#897
asayyah wants to merge 1 commit intomasterfrom
lint-config-update

Conversation

@asayyah
Copy link
Contributor

@asayyah asayyah commented Mar 14, 2026

Summary

  • Migrate deprecated dtls.Client/dtls.Listen/dtls.Config to dtls.ClientWithOptions/dtls.ListenWithOptions with functional options (staticcheck SA1019)
  • Suppress gosec G115 for intentional uint32→byte truncation in renomination serialization

Test plan

  • golangci-lint run ./... passes with 0 issues (v2.10.1)
  • go build ./... succeeds

@codecov
Copy link

codecov bot commented Mar 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.49%. Comparing base (bca1b4c) to head (84445d0).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #897      +/-   ##
==========================================
+ Coverage   88.40%   88.49%   +0.08%     
==========================================
  Files          44       44              
  Lines        5591     5591              
==========================================
+ Hits         4943     4948       +5     
+ Misses        448      444       -4     
+ Partials      200      199       -1     
Flag Coverage Δ
go 88.49% <100.00%> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@asayyah asayyah requested a review from JoTurk March 14, 2026 00:42
@JoTurk
Copy link
Member

JoTurk commented Mar 14, 2026

Thank you, this should be done inside the bot branch here #895 also to fix the modernize issues.

@asayyah asayyah closed this Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants