-
Notifications
You must be signed in to change notification settings - Fork 29
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
Polish tests #65
Polish tests #65
Conversation
* Update dependency com.github.siom79.japicmp:japicmp to v0.17.3 * Polish IncompatibleChangeExclusionTest --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Goooler <wangzongler@gmail.com>
task japicmpDetectsNewDefaultMethodByDefault(type: me.champeau.gradle.japicmp.JapicmpTask) { | ||
oldClasspath.from(jar) | ||
newClasspath.from(jarv2) | ||
failOnModification = true | ||
failOnSourceIncompatibility = true | ||
} | ||
|
||
task configuredToIgnoreNewDefaultMethod(type: me.champeau.gradle.japicmp.JapicmpTask) { | ||
oldClasspath.from(jar) | ||
newClasspath.from(jarv2) | ||
failOnModification = true | ||
failOnSourceIncompatibility = true | ||
compatibilityChangeExcludes = [ "METHOD_NEW_DEFAULT" ] | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See siom79/japicmp#289.
private final static GradleVersion GRADLE_MIN = GradleVersion.version("6.0"); | ||
private final static GradleVersion GRADLE_MIN = GradleVersion.version("6.6"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some APIs like Provider.zip
depend on Gradle 6.6.
japicmp-gradle-plugin/src/main/java/me/champeau/gradle/japicmp/report/RichReport.java
Lines 114 to 116 in 716fe56
public Provider<RegularFile> getOutputFile() { | |
return getDestinationDir().zip(getReportName(), Directory::file); | |
} |
Gradle intro configuration cache in 6.6, we are now on it, see https://docs.gradle.org/6.6/release-notes.html#configuration-caching. But I also notified some CC compatibility issues left in c398274, todo it.
3d8c880
to
bf6bee3
Compare
Thanks for the update! |
Follow up #63.