chore(ci): add linter config for gofumpt
#4929
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4929 +/- ##
==========================================
+ Coverage 75.86% 78.01% +2.15%
==========================================
Files 470 540 +70
Lines 37301 31271 -6030
==========================================
- Hits 28299 24397 -3902
+ Misses 7071 4832 -2239
- Partials 1931 2042 +111 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
This PR adds linter configuration for gofumpt with extra rules enabled, which enforces stricter Go formatting standards. The changes apply gofumpt's formatting rules across the codebase, primarily simplifying type parameter declarations by removing redundant type repetitions.
Key changes:
- Added
gofumptlinter configuration withextra-rules: truein.golangci.yml - Simplified function signatures and type parameter declarations where consecutive parameters share the same type
- Applied formatting to function parameters, generic type constraints, and anonymous function signatures
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| .golangci.yml | Added gofumpt linter configuration with extra-rules enabled |
| snapshot/upload/upload_progress.go | Simplified interface method signature with consecutive int64 parameters |
| repo/blob/sharded/sharded.go | Simplified anonymous function signature with consecutive string parameters |
| internal/stats/count_map.go | Simplified anonymous function signature with consecutive any parameters |
| internal/metrics/metrics_timeseries.go | Simplified generic interface method signature with repeated type parameter |
| internal/contentlog/contentlog_logger.go | Simplified generic type constraints across multiple logging functions |
| cli/command_benchmark.go | Simplified generic type parameter declaration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.