Skip to content

Update to gradle v9#7590

Merged
jkwatson merged 5 commits intoopen-telemetry:mainfrom
jaydeluca:gradle-9-upgrade
Aug 26, 2025
Merged

Update to gradle v9#7590
jkwatson merged 5 commits intoopen-telemetry:mainfrom
jaydeluca:gradle-9-upgrade

Conversation

@jaydeluca
Copy link
Member

@jaydeluca jaydeluca commented Aug 18, 2025 •
edited
Loading

Supersedes #7523

@jaydeluca
chore(deps): update dependency gradle to v9
25af009 
- Update gradle wrapper to 9.0.0
- Fix gradle configuration issues in build files
- Remove redundant configuration.add() calls in animal-sniffer-signature
- Update protobuf gradle plugin dependency
- Fix ProjectDependency comparison in custom-checks
@jaydeluca
dont refernce project variables in dolast, exclude protos from checks…
3a3348e 
…tyle
@jaydeluca
try different approach for exclusion
9d0d8dc
@codecov
Copy link

codecov bot commented Aug 20, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.99%. Comparing base (18aca41) to head (4f760ef).
⚠️ Report is 14 commits behind head on main.

Additional details and impacted files 
@@             Coverage Diff              @@
##               main    #7590      +/-   ##
============================================
- Coverage     90.01%   89.99%   -0.02%     
+ Complexity     7080     7079       -1     
============================================
  Files           803      803              
  Lines         21417    21419       +2     
  Branches       2086     2086              
============================================
- Hits          19278    19276       -2     
- Misses         1477     1479       +2     
- Partials        662      664       +2

☔ View full report in Codecov by Sentry.
📢 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.

@jaydeluca jaydeluca changed the title update dependency gradle to v9 Update to gradle v9 Aug 20, 2025
@jaydeluca jaydeluca marked this pull request as ready for review August 20, 2025 20:04
@jaydeluca jaydeluca requested a review from a team as a code owner August 20, 2025 20:04
trask
trask approved these changes Aug 20, 2025
View reviewed changes
Copy link
Member

@trask trask left a comment

Choose a reason for hiding this comment

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

Thanks!

@jaydeluca @trask
Update build.gradle.kts
4f760ef 
Co-authored-by: Trask Stalnaker <[email protected]>
breedx-splk
breedx-splk approved these changes Aug 22, 2025
View reviewed changes
Copy link
Contributor

@breedx-splk breedx-splk left a comment

Choose a reason for hiding this comment

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

Amazing! Thanks @jaydeluca. Curious if you tried any of the publish work (like even ./gradlew publishToMavenLocal)?

@jaydeluca
Copy link
Member Author

jaydeluca commented Aug 22, 2025

@breedx-splk yes publishToMavenLocal looks good for me locally, are there others I would be able to test that you think would be useful?

@breedx-splk
Copy link
Contributor

breedx-splk commented Aug 25, 2025

@breedx-splk yes publishToMavenLocal looks good for me locally, are there others I would be able to test that you think would be useful?

Not off the top of my head -- just thinking about potential edge cases that don't get flexed in the normal PR and main merge builds. Thanks again!

jkwatson
jkwatson approved these changes Aug 26, 2025
View reviewed changes
Copy link
Contributor

@jkwatson jkwatson left a comment

Choose a reason for hiding this comment

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

Thanks!

@jkwatson jkwatson merged commit 21c1b02 into open-telemetry:main Aug 26, 2025
29 checks passed
the-clam pushed a commit to the-clam/opentelemetry-java that referenced this pull request Nov 6, 2025
@jaydeluca @trask @the-clam
Update to gradle v9 (open-telemetry#7590)
488f5f2 
Co-authored-by: Trask Stalnaker <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@trask trask trask approved these changes

@jkwatson jkwatson jkwatson approved these changes

@breedx-splk breedx-splk breedx-splk approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jaydeluca @breedx-splk @trask @jkwatson