Skip to content

chore: Replace deprecated gofumpt extra-rules with extra options - #9062

Merged
pellared merged 2 commits into
open-telemetry:mainfrom
kd-collective:remove-gofumpt-extra-rules
Oct 1, 2026
Merged

pellared merged 2 commits into
open-telemetry:mainfrom
kd-collective:remove-gofumpt-extra-rules

Conversation

@kijimad

@kijimad kijimad commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #8942. make lint still warns because extra-rules: true was kept:

[formatter] gofumpt: `extra-rules` is deprecated, please use `extra.group-params` instead.

extra-rules: true currently enables all three gofumpt extra rules (gofumpt v0.12.0).

Following the warning as-is would turn off clothe-returns and balance-calls, so all three are listed to keep formatting unchanged.

Testing

  • make lint no longer prints the deprecation warning, and all modules report 0 issues.

@codecov

codecov Bot commented Oct 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.1%. Comparing base (80f84f5) to head (350b3f5).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #9062     +/-   ##
=======================================
- Coverage   89.1%   89.1%   -0.1%     
=======================================
  Files        338     338             
  Lines      22274   22274             
=======================================
- Hits       19855   19853      -2     
- Misses      2419    2421      +2     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dmathieu dmathieu added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Oct 1, 2026
@pellared

pellared commented Oct 1, 2026

Copy link
Copy Markdown
Member

@kijimad, do you want to make the same PR in https://github.com/open-telemetry/opentelemetry-go-contrib ?

@pellared
pellared merged commit ef11bfc into open-telemetry:main Oct 1, 2026
50 of 51 checks passed
@kijimad

kijimad commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

@pellared
Sure, I'll open one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants