-
Notifications
You must be signed in to change notification settings - Fork 55
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
feat: Add Maven release #1111
feat: Add Maven release #1111
Conversation
# Configure the docker image for kokoro-trampoline. | ||
env_vars: { | ||
key: "TRAMPOLINE_IMAGE" | ||
value: "gcr.io/cloud-devrel-kokoro-resources/java8" |
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.
I know we had internal discussions about java8 vs java11 for this image. No preference from my end in this PR, but this is something to keep in mind in the gapic monorepo.
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.
Unlike the other repos where we can compile with Java 11 and make the source compatible with Java 8, we have to compile gapic-generator-java in Java 8. It's not a dependency to client libraries, it has to load classes from Java 8 instead of Java 11 during compilation time.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
.kokoro
folder for releasing gapic-generaotr-java as a jar..kokoro
folder in gax-java, and only keeping the necessary.simple
tojava-yoshi