Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 3, 2025

  • Add otel.japicmp-conventions to otel.publish-conventions.gradle.kts to apply it centrally
  • Remove otel.japicmp-conventions from individual module build files
  • Update otel.japicmp-conventions to skip projects without a jar task (like BOMs)
  • Test that japicmp still runs correctly for stable modules
  • Revert unnecessary formatting changes to minimize diff
Original prompt

This section details on the original issue you should resolve

<issue_title>Try to add otel.japicmp-conventions centrally</issue_title>
<issue_description>Instead of adding it to each stable module.

Check out how this is done in core (SDK) repo: https://github.com/search?q=repo%3Aopen-telemetry%2Fopentelemetry-java%20otel.japicmp-conventions&type=code

Originally posted by @trask in #11763 (review)
</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #11794

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Added otel.japicmp-conventions to otel.publish-conventions.gradle.kts to apply it centrally
- Removed explicit otel.japicmp-conventions from individual stable module build files
- Updated japicmp-conventions to only apply to projects with a jar task (excludes BOMs/platforms)
- Applied spotless formatting fixes to maintain consistent code style

Co-authored-by: trask <[email protected]>
Copilot AI changed the title [WIP] Try to add otel.japicmp-conventions centrally Apply otel.japicmp-conventions centrally in otel.publish-conventions Oct 3, 2025
Copilot AI requested a review from trask October 3, 2025 01:22
Copilot finished work on behalf of trask October 3, 2025 01:22
@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Oct 3, 2025
Copilot AI requested a review from trask October 3, 2025 03:18
Copilot finished work on behalf of trask October 3, 2025 03:18
Copy link
Member

@trask trask Oct 3, 2025

Choose a reason for hiding this comment

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

the changes in this file makes a lot more sense when you select "hide whitespace" from the gear icon above (because of the extra level of indentation introduced)

@trask trask marked this pull request as ready for review October 3, 2025 04:03
@trask trask requested a review from a team as a code owner October 3, 2025 04:03
@trask trask merged commit d2b7dbc into main Oct 3, 2025
97 of 181 checks passed
@trask trask deleted the copilot/fix-8ac7d4d7-82ae-43cc-9c7a-7d7deeab2b7a branch October 3, 2025 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test native This label can be applied to PRs to trigger them to run native tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Try to add otel.japicmp-conventions centrally

3 participants