Skip to content

chore: upgrade golangci-lint#665

Open
marcopaggioro wants to merge 14 commits into
mainfrom
chore/upgrade-golangci
Open

chore: upgrade golangci-lint#665
marcopaggioro wants to merge 14 commits into
mainfrom
chore/upgrade-golangci

Conversation

@marcopaggioro
Copy link
Copy Markdown
Member

@marcopaggioro marcopaggioro commented May 14, 2026

Summary 💡

This PR aims to upgrade the version of golangci-lint.
This is required to alow the upgrade to the next major of version partially done in #662 .

Description 📝

Docs:

Core activities:

  • Moved functions and structs to comply with best practice order
  • Filled non-exaustive switches
  • Renamed packages with "reserved" name with "x" as suffix (like template -> templatex) as already done in previous packages
  • Syntax improvement like slices.Contains
  • Fix comments
  • Removed unused functions and consts
  • Use testing context in tests

Notes:

  • .golangci.yml syntax changes from the 1.x version to the 2.x version. Version 2.x offers a cli utility and a dedicated docs page to help migration.
  • Foreach new version I read the changelog and aligned the new lints with the golintci "golden config", where possible.

Breaking Changes 💔

Not detected.

Tests performed 🧪

All test suites.

Future work 🔧

Finish the Go major upgrade in #662

@marcopaggioro marcopaggioro force-pushed the chore/upgrade-golangci branch from 1b11d9d to 25a5e36 Compare May 14, 2026 13:01
@marcopaggioro marcopaggioro force-pushed the chore/upgrade-golangci branch from 25a5e36 to a3ddf8b Compare May 19, 2026 14:17
@marcopaggioro marcopaggioro force-pushed the chore/upgrade-golangci branch from a3ddf8b to 67dc21c Compare May 19, 2026 14:38
@marcopaggioro marcopaggioro self-assigned this May 19, 2026
@marcopaggioro marcopaggioro marked this pull request as ready for review May 19, 2026 15:00
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.

1 participant