-
Notifications
You must be signed in to change notification settings - Fork 5k
Transform precompiled plugins sample to publishing convention plugins sample #14241
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
Conversation
… sample Focus on the multi-repo use case and demonstrate how convention plugins can be published and consumed from another project.
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.
Well done! I like the sample a lot.
I found some smallish things.
There are leftover setting files in the subprojects that should be removed:
- convention-plugins/settings.gradle(.kts)
subprojects/docs/src/samples/build-organization/publishing-convention-plugins/README.adoc
Show resolved
Hide resolved
subprojects/docs/src/samples/build-organization/publishing-convention-plugins/README.adoc
Outdated
Show resolved
Hide resolved
subprojects/docs/src/samples/build-organization/publishing-convention-plugins/README.adoc
Outdated
Show resolved
Hide resolved
subprojects/docs/src/samples/build-organization/publishing-convention-plugins/README.adoc
Outdated
Show resolved
Hide resolved
subprojects/docs/src/samples/build-organization/publishing-convention-plugins/README.adoc
Outdated
Show resolved
Hide resolved
subprojects/docs/src/samples/build-organization/publishing-convention-plugins/README.adoc
Outdated
Show resolved
Hide resolved
subprojects/docs/src/samples/build-organization/publishing-convention-plugins/README.adoc
Outdated
Show resolved
Hide resolved
...ples/build-organization/publishing-convention-plugins/groovy/convention-plugins/build.gradle
Outdated
Show resolved
Hide resolved
// tag::plugins[] | ||
plugins { | ||
id("myproject.java-conventions") | ||
id("com.myorg.java-conventions") |
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.
The integration test setup below does not follow what we would recommend today. Following what we do here:
https://github.com/gradle/gradle/blob/418a06a84863e20bebcdebc924761e6312d53626/subprojects/docs/src/samples/java/modules-multi-project-with-integration-tests/kotlin/buildSrc/src/main/kotlin/myproject.java-conventions.gradle.kts
It could look like this:
val integrationTest by sourceSets.creating
configurations[integrationTest.implementationConfigurationName].extendsFrom(configurations.testImplementation.get())
configurations[integrationTest.runtimeOnlyConfigurationName].extendsFrom(configurations.testRuntimeOnly.get())
val integrationTestTask = tasks.register<Test>("integrationTest") {
testClassesDirs = integrationTest.output.classesDirs
classpath = integrationTest.runtimeClasspath
shouldRunAfter(tasks.test)
}
dependencies {
"integrationTestImplementation"(project)
}
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
…vention-plugins/README.adoc Co-authored-by: Jendrik Johannes <[email protected]>
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.
Looks great.
I only have a suggestion to shorten title and description so that it fits into one line when rendered.
Co-authored-by: Jendrik Johannes <[email protected]>
Co-authored-by: Jendrik Johannes <[email protected]>
Focus on the multi-repo use case and demonstrate how convention plugins can be published and consumed from another project.