Skip to content

Update spotless packages to v8.11.0 - #9002

Merged
jack-berg merged 6 commits into
mainfrom
renovate/spotless-packages
Nov 9, 2026
Merged

jack-berg merged 6 commits into
mainfrom
renovate/spotless-packages

Conversation

@renovate

@renovate renovate Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
com.diffplug.spotless:spotless-plugin-gradle 8.10.4 → 8.11.0 age confidence
com.diffplug.spotless 8.10.4 → 8.11.0 age confidence

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • Between 12:00 AM and 07:59 AM, only on Tuesday (* 0-7 * * 2)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate
renovate Bot requested a review from a team as a code owner November 3, 2026 07:36
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-11-09 18:08 UTC

Status above doesn't look right?
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@jack-berg

Copy link
Copy Markdown
Member

FYI, this version of spotless brings over a new column width. The result is that most java files are reformatted, so big diff. I rather like the change as it makes the code more compact (as evidence by the net 1700 line reduction) without jeopardizing readability IMO.

But let's talk about whether we want this @open-telemetry/java-approvers. The alternative is to revert to the previous column width.

@renovate

renovate Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@jkwatson

jkwatson commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

FYI, this version of spotless brings over a new column width. The result is that most java files are reformatted, so big diff. I rather like the change as it makes the code more compact (as evidence by the net 1700 line reduction) without jeopardizing readability IMO.

But let's talk about whether we want this @open-telemetry/java-approvers. The alternative is to revert to the previous column width.

It looks like a good change. I'm in favor (but I barely contribute any more, so my opinion should be not considered strongly)

@jaydeluca

Copy link
Copy Markdown
Member

i like the changes

@trask trask left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it too and planning to accept the change in the instrumentation repo as well (it's net loss of 8k LOC in that repo 😅)

@jack-berg
jack-berg enabled auto-merge (squash) November 9, 2026 14:49
@codecov

codecov Bot commented Nov 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.86364% with 49 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.37%. Comparing base (f02bddc) to head (e2815b3).

Files with missing lines Patch % Lines
...orter/sender/okhttp/internal/OkHttpGrpcSender.java 65.62% 6 Missing and 5 partials ⚠️
...etry/exporter/internal/otlp/KeyValueMarshaler.java 0.00% 7 Missing ⚠️
.../api/baggage/propagation/W3CBaggagePropagator.java 82.60% 2 Missing and 2 partials ⚠️
...arativeconfig/YamlDeclarativeConfigProperties.java 69.23% 1 Missing and 3 partials ⚠️
...ry/exporter/otlp/internal/GrpcExporterBuilder.java 50.00% 3 Missing ⚠️
...ry/exporter/otlp/internal/HttpExporterBuilder.java 50.00% 3 Missing ⚠️
...ativeconfig/ComposableRuleBasedSamplerFactory.java 92.59% 0 Missing and 2 partials ⚠️
...xport/LegacyLogRecordProcessorInstrumentation.java 33.33% 2 Missing ⚠️
...etry/sdk/logs/export/SimpleLogRecordProcessor.java 84.61% 1 Missing and 1 partial ⚠️
.../internal/state/DeltaSynchronousMetricStorage.java 93.10% 0 Missing and 2 partials ⚠️
... and 8 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9002      +/-   ##
============================================
+ Coverage     90.35%   90.37%   +0.02%     
- Complexity    11017    11018       +1     
============================================
  Files          1021     1021              
  Lines         30128    30124       -4     
  Branches       3933     3933              
============================================
+ Hits          27221    27225       +4     
+ Misses         1953     1941      -12     
- Partials        954      958       +4     

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

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

@jack-berg
jack-berg merged commit fecfb6d into main Nov 9, 2026
34 checks passed
@jack-berg
jack-berg deleted the renovate/spotless-packages branch November 9, 2026 18:07
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.

4 participants