Skip to content

Conversation

@janos
Copy link
Member

@janos janos commented Oct 5, 2025

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

This PR updates the code only in formal way according to the language and standard library improvements over the years. It contains mainly the changes that go run golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize@latest -fix ./... command made and what linters suggested.

These changes should, and must, not change any existing behaviour.

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

General code maintenance.

Related Issue (Optional)

Screenshots (if appropriate):

@gacevicljubisa
Copy link
Member

gacevicljubisa commented Oct 6, 2025

This is great. It will prevent many of minor PRs, and it will reduce tme spent on checking on them. To bad that gopls is not provided by golangci-lint. Also it improves the code.

@gacevicljubisa gacevicljubisa self-requested a review October 6, 2025 07:07
@gacevicljubisa gacevicljubisa changed the title chore: modernize codebaze for go 1.25 chore: modernize codebase for go 1.25 Oct 6, 2025
@gacevicljubisa gacevicljubisa merged commit a74befd into master Oct 6, 2025
22 of 29 checks passed
@gacevicljubisa gacevicljubisa deleted the modernize-go-1.25 branch October 6, 2025 12:41
janos added a commit that referenced this pull request Oct 15, 2025
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.

3 participants