Skip to content

Conversation

@nrwiersma
Copy link
Contributor

@nrwiersma nrwiersma commented Dec 15, 2025

Goal of this PR

This bumps the linter version.

How did I test it?

@nrwiersma nrwiersma self-assigned this Dec 15, 2025
Copilot AI review requested due to automatic review settings December 15, 2025 12:23
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR bumps the golangci-lint version and applies code improvements suggested by the linter, primarily modernizing code to use newer Go standard library functions.

  • Upgrades golangci-lint from v2.4.0 to v2.7.2
  • Replaces manual map copying with maps.Copy from the standard library
  • Replaces strings.HasPrefix + strings.TrimPrefix pattern with more efficient strings.CutPrefix

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/test.yml Updates the golangci-lint version environment variable
op.go Refactors map initialization and copying to use maps.Copy with proper capacity allocation
cmd/oapi-gen/gen.go Simplifies string prefix checking using strings.CutPrefix

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nrwiersma nrwiersma merged commit 434bc41 into main Dec 15, 2025
21 checks passed
@nrwiersma nrwiersma deleted the nw/maintenance branch December 15, 2025 12:39
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.

4 participants