Skip to content

Fix prealloc lint warning in test#141

Merged
DXTimer merged 1 commit intomainfrom
fix/lint-prealloc
Mar 18, 2026
Merged

Fix prealloc lint warning in test#141
DXTimer merged 1 commit intomainfrom
fix/lint-prealloc

Conversation

@DXTimer
Copy link
Copy Markdown
Contributor

@DXTimer DXTimer commented Mar 18, 2026 •
edited
Loading

Fix prealloc lint finding by preallocating msgAnswers slice with the known capacity from rec.Msg.Answer. This was the only lint issue preventing make release from succeeding.

📋 Deployment Pre/Post tasks

N/A

:shipit: Deployment Verification

  • golangci-lint run passes with 0 issues
  • go test ./... passes

@DXTimer
Fix prealloc lint warning in test
2fc7f5f 
Preallocate msgAnswers slice with known capacity from rec.Msg.Answer.
@DXTimer DXTimer self-assigned this Mar 18, 2026
@DXTimer DXTimer added the bug Code defect or incorrect behavior label Mar 18, 2026
@DXTimer DXTimer requested a review from a team March 18, 2026 06:08
@DXTimer DXTimer merged commit d2ced7c into main Mar 18, 2026
2 checks passed
@DXTimer DXTimer deleted the fix/lint-prealloc branch March 18, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NelsonVides NelsonVides NelsonVides approved these changes

Assignees

@DXTimer DXTimer

Labels

bug Code defect or incorrect behavior

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DXTimer @NelsonVides