Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore METHOD_NEW_DEFAULT binary compatibility #3181

Closed
shakuzen opened this issue May 16, 2022 · 1 comment
Closed

Ignore METHOD_NEW_DEFAULT binary compatibility #3181

shakuzen opened this issue May 16, 2022 · 1 comment
Labels
build A change in our build-system type: task A general task
Milestone

Comments

@shakuzen
Copy link
Member

We check binary compatibility in the build against the previous version to catch incompatibilities introduced. This catches new default methods on interfaces, which we don't want blocking the build, since we expect this to be compatible. This came up in #3175 (comment). It was brought up in the japicmp repo in siom79/japicmp#201. However, it looks like we're missing a straightforward way to configure accepting METHOD_NEW_DEFAULT changes in the Gradle plugin. It has been requested in melix/japicmp-gradle-plugin#43. We may have to do a temporary workaround until that is resolved.

@shakuzen shakuzen added type: task A general task build A change in our build-system labels May 16, 2022
@shakuzen shakuzen added this to the 1.10 backlog milestone May 16, 2022
shakuzen added a commit to shakuzen/micrometer that referenced this issue May 16, 2022
Manually exclude the new method since we cannot easily exclude the new default method check in the current version of the gradle plugin. See micrometer-metricsgh-3181.
@shakuzen shakuzen modified the milestones: 1.10 backlog, 1.8.x Sep 12, 2022
@shakuzen
Copy link
Member Author

We can now do this more easily thanks to melix/japicmp-gradle-plugin#43 being implemented. I've also updated it to target 1.8.x since japicmp was backported there since this issue was opened.

izeye added a commit to izeye/micrometer that referenced this issue Sep 27, 2022
This commit also replaces method excludes with METHOD_NEW_DEFAULT compatibility change exclude.

Closes micrometer-metricsgh-3181
@shakuzen shakuzen modified the milestones: 1.8.x, 1.8.11 Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build A change in our build-system type: task A general task
Projects
None yet
Development

No branches or pull requests

1 participant