[#177] feat(CI): Support publish Gravitino related jars and docs to OSSRH #543
Conversation
Code Coverage Report
Files
|
|
Let's wait for others to be merged first. |
jerryshao
changed the title
| @@ -42,6 +42,10 @@ java { | |||
| } | |||
| } | |||
|
|
|||
| tasks.named("generateMetadataFileForMavenJavaPublication") { | |||
There was a problem hiding this comment.
Where will the generateMetadataFileForMavenJavaPublication task be called? and this task name is too long.
There was a problem hiding this comment.
It's one of the publishing tasks. Please see below:
There was a problem hiding this comment.
The task name can be simplified, seems to long for now.
There was a problem hiding this comment.
Should we rename it as the following?
tasks.named("generateMetadataFileForMavenJavaPublication").configure {
setName("newTaskName")
}Personally, I do not see the need to change the default name for publishing tasks.
There was a problem hiding this comment.
So you mean this name "generateMetadataFileForMavenJavaPublication" is predefined in the publish task?
There was a problem hiding this comment.
Yes, all tasks in the picture above were defined by the publishing plugin NOT me, I was even not aware this task before this changes.
xunliu
changed the title
…SSRH (#543) ### What changes were proposed in this pull request? - Add the Kotlin script in the root `build.gradle.kts` to publish Gravitino to maven central repo - Fix several Java doc issues. ### Why are the changes needed? Fix: #177 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? N/A
What changes were proposed in this pull request?
build.gradle.ktsto publish Gravitino to maven central repoWhy are the changes needed?
Fix: #177
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
N/A