🧹 [Maintenance]: Update golangci-lint #2816
Description
Maintenance Task Description
As of writing, the latest version of golangci-lint is v1.55.2. This issue encompasses updating the github actions to use at least this version of golangci-lint along with updating the .golanci.yml config to enable or disable new rules and linters as agreed upon by the fiber team.
- gosmopolitan checks for i18n usage, including time.Local
- mirror checks for allocs in string/[]byte conversions
- tagalign checks for struct tag formatting
- gochecksumtype checks for exhaustive type switches. Somewhat complex, check out the examples.
- inamedparam requires named params for interface functions (can exclude single param funcs)
- perfsprint checks faster alternatives to fmt.Sprintf
- testifylint checks idiomatic testify usage
- zerologlint zerolog usage
- sloglint slog usage
- protogetter protobuf usage
In addition to the new linters, there are several new linter rules for existing linters, many of which raise errors in the current codebase. These will likely need to be inspected individually.
Impact on the Project
Updating the linter benefits the project by keeping up with modern Go practices, maintaining code consistency, and reducing the probability of bugs and API issues.
Additional Context (optional)
No response
Checklist:
- I have confirmed that this maintenance task is currently not being addressed.
- I understand that this task will be evaluated by the maintainers and prioritized accordingly.
- I am available to provide further information if needed.