Skip to content

Conversation

eskatos
Copy link
Member

@eskatos eskatos commented Nov 12, 2023

There is some classloading gymnastic involved in order for the build-logic/ tasks to use the right version of gradle-kotlin-dsl-shared-runtime without requiring a wrapper update and to isolate source code parsing for @since tags from gradle-kotlin-dsl-shared-runtime.

…uction code instead of code from the wrapper

by isolating the execution in a tailored classloader
via a JDK only facade
when the extended type itself is `@Incubating`
by specifying explicit return types where ambiguous
by removing unnecessary type parameter
by fixing java.lang.annotation.Annotation package name in mapped type strings
by using URLClassloader instead of VisitableURLClassLoader
by parsing java sources to get the javadoc tag value
@eskatos eskatos added this to the 8.6 RC1 milestone Nov 12, 2023
@eskatos eskatos self-assigned this Nov 12, 2023
by fixing indentation to make ktlint happy
@eskatos eskatos requested a review from jbartok November 12, 2023 18:39
@eskatos eskatos marked this pull request as ready for review November 12, 2023 18:39
@eskatos eskatos requested a review from a team November 12, 2023 18:39
@eskatos eskatos requested a review from a team as a code owner November 12, 2023 18:39
Copy link
Member

@jbartok jbartok left a comment

Choose a reason for hiding this comment

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

LGTM

@eskatos
Copy link
Member Author

eskatos commented Nov 14, 2023

@bot-gradle merge

@bot-gradle
Copy link
Collaborator

Your PR is queued. See the queue page for details.

@bot-gradle
Copy link
Collaborator

I've triggered a build for you. Click here to see all failures if there's any.

@bot-gradle bot-gradle merged commit e4b8411 into master Nov 14, 2023
@bot-gradle bot-gradle deleted the eskatos/kotlin-dsl/api-extensions-incubating-since branch November 14, 2023 11:26
meowool-bot added a commit to meowool/cradle that referenced this pull request Nov 14, 2023
This commit synchronizes the latest 9 changes from the https://github.com/gradle/gradle/tree/master. The changes include:

e4b8411: gradle/gradle#27025 Let generated Kotlin extensions for the Gradle API be properly annotated with `@Incubating` and `@since`
f887f85: Polish ApiExtensionsGenerator
8a34f3f: Add `@since` to generated Kotlin extensions
791e380: Upgrade qdox from 2.0.0 to 2.0.3 in build-logic/
afe4fef: Polish KotlinExtensionsForGradleApiFacade
3160d53: Polish ApiTypeProvider
8c88929: Polish ApiExtensionsGenerator
9768a05: Let Gradle API Kotlin extension files be annotated `@Incubating`
4ae3f54: Rework GenerateKotlinExtensionsForGradleApi task to actually use production code instead of code from the wrapper
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding a new public type and a method fails binary compatibility check

3 participants