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
Release 0.15.1 differs from 0.14.4, treats new annotation file as incompatible #279
Comments
Attached the reports from both japicmp versions. The only change between runs was the depended version. |
Is 0.15.x going to be on Maven Central? |
@garydgregory Yes, japicmp 0.15.1 is available on Maven Central. @jhy After upgrading all the dependencies one test case failed and I "fixed" it. But it looks like the "fix" introduced another issue. The japicmp 0.15 report shows the change |
Fixed: 0.15.2 |
Awesome, thanks Martin! With this fix and the fix for JDK 16-ea, I've been able to enable CI builds in jsoup for 16-ea again. |
Thanks. Great to hear that.
Jonathan Hedley <notifications@github.com> schrieb am Sa., 9. Jan. 2021,
07:12:
… Awesome, thanks Martin!
With this fix and the fix for JDK 16-ea, I've been able to enable CI
builds in jsoup for 16-ea again.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#279 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA4U7NAJ7IUAX6QQHSQWXLTSY7XV5ANCNFSM4VO45GSA>
.
|
Thanks for releasing 0.15.1, I was just moving jsoup to it (from 0.14.4) but found that the new version reports a new annotation as a breaking change (for source compatibility).
Version 14.4:
Version 15.1:
The actual new annotation is at https://github.com/jhy/jsoup/blob/master/src/main/java/org/jsoup/internal/FieldsAreNonnullByDefault.java . I believe (hope!) this is a binary and source compatible change?
In the full XML report, every line item for the class is marked as both source and binary compatible, in both versions.
Also, in the HTML report, in 15 it is marked source incompatible with a (*), but the text is green. I expected it to be red, and it took a while to find it.
Is this a bug from japicmp updating its dependencies, or is this an actual source compatibility issue that I've misunderstood?
Thanks,
Jonathan
The text was updated successfully, but these errors were encountered: