Skip to content

Conversation

jaydeluca
Copy link
Member

@jaydeluca jaydeluca commented Aug 18, 2025

Supersedes #7523

- 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
Copy link

codecov bot commented Aug 20, 2025

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
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!

Co-authored-by: Trask Stalnaker <[email protected]>
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

@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 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!

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
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