Walkthrough

Upgrade Makefile lint invocation to the golangci-lint v2 module path and version; adjust several tests to use Empty assertions and simplify nolint directives in convert.go. No functional behavior or public API changes.

Changes

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Pay attention to test assertion intent in cbor_test.go, common_test.go, and http_test.go to ensure semantic equivalence.
• Verify the Makefile module path/version change works in CI and local tooling.

Possibly related PRs

• chore: update golangci-lint to v2 contrib#1445 — Also migrates to golangci-lint v2 module path/version and adjusts lint-related code.
• Bump golangci-lint to v1.61.0 fiber#3135 — Similar Makefile golangci-lint bump; related to lint tool updates.
• Bump golangci-lint to v1.59.1 #87 — Updates Makefile lint target to a newer golangci-lint version; related dependency update pattern.

Suggested reviewers

• gaby
• sixcolors
• efectn

Poem

🐇 I nibbled at warnings, hopped through each test,

Switched a lint path, and cleaned up the rest.
Empty assertions snug in their place,
A tidy Makefile, quick little race.
🥕✨

Pre-merge checks and finishing touches

Learnt from: CR
Repo: gofiber/fiber PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-10-22T13:48:12.485Z
Learning: Run golangci-lint (Make target: lint)

Learnt from: CR
Repo: gofiber/fiber PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-07-19T15:13:10.652Z
Learning: Run `make lint` for static analysis when modifying code

Learnt from: efectn
Repo: gofiber/fiber PR: 3162
File: hooks_test.go:228-228
Timestamp: 2024-12-13T08:14:22.851Z
Learning: In Go test files, prefer using the `require` methods from the `testify` package for assertions instead of manual comparisons and calls to `t.Fatal` or `t.Fatalf`.

Learnt from: ReneWerner87
Repo: gofiber/fiber PR: 3161
File: app.go:923-932
Timestamp: 2024-11-15T07:56:21.623Z
Learning: In the Fiber framework, breaking changes are acceptable when moving from version 2 to version 3, including modifications to method signatures such as in the `Test` method in `app.go`.

Learnt from: CR
Repo: gofiber/cli PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-09-03T12:18:29.841Z
Learning: Always run `make lint`

Learnt from: efectn
Repo: gofiber/fiber PR: 3162
File: hooks_test.go:228-228
Timestamp: 2024-12-13T08:14:22.851Z
Learning: In Go test files, prefer using the `require` methods from the `testify` package for assertions instead of manual comparisons and calls to `t.Fatal` or `t.Fatalf`.

Learnt from: grivera64
Repo: gofiber/fiber PR: 3807
File: adapter_test.go:118-144
Timestamp: 2025-10-16T07:19:52.418Z
Learning: In the Fiber codebase, the linter does not allow `require` assertions from within HTTP handlers (including net/http-style handlers). Use `t.Fatalf`, `t.Errorf`, or similar `testing.T` methods for error handling inside handler functions instead.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Summary of Changes

Hello @ReneWerner87, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request significantly upgrades the project's linting infrastructure by moving to the latest major version of golangci-lint. This update includes a complete migration of the linting configuration to the new schema, integrates Go formatters directly into the linting process, and refines various linter settings. The goal is to enhance code quality, enforce modern Go best practices, and streamline the development workflow by ensuring consistent code style and catching potential issues earlier.

Highlights

• golangci-lint Version Bump: The golangci-lint tool has been updated to version v2.6.1.
• Configuration Migration: The .golangci.yml configuration has been migrated to the new v2 schema, involving structural changes to settings and exclusions.
• Formatter Integration: Dedicated formatter configuration for gofmt and gofumpt has been added, ensuring consistent code styling.
• Linter Settings Adjustment: Specific settings for revive, staticcheck, and testifylint have been tuned for compatibility and improved linting results.
• Makefile Update: The Makefile has been updated to reflect the new golangci-lint version in the lint command.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.31%. Comparing base (9417f80) to head (f8d6b83).
⚠️ Report is 6 commits behind head on master.

@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   83.31%   83.31%           
=======================================
  Files          11       11           
  Lines         755      755           
=======================================
  Hits          629      629           
  Misses        115      115           
  Partials       11       11           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.