uranusjr commented on code in PR #67826:
URL: https://github.com/apache/airflow/pull/67826#discussion_r3333046463
##########
.github/workflows/publish-docs-to-s3.yml:
##########
@@ -345,8 +345,53 @@ jobs:
if-no-files-found: 'error'
overwrite: 'true'
+ build-java-sdk-docs:
+ needs: [build-info]
+ timeout-minutes: 30
+ name: "Build Java SDK Javadoc"
+ runs-on: ubuntu-latest
+ permissions:
+ contents: read
+ steps:
+ - name: "Checkout ${{ inputs.ref }}"
+ uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd #
v6.0.2
+ with:
+ persist-credentials: false
+ ref: ${{ inputs.ref }}
+ # Keep these in sync:
+ # - jvmTarget, languageVersion, and sourceCompatibility in
java-sdk/build.gradle.kts
+ # - TEMURIN_VERSION in scripts/docker/install_jdk.sh
+ # - JAVA_VERSION in .github/workflows/ci-amd.yml and
.github/workflows/ci-arm.yml
+ # - java-version in .github/workflows/codeql-analysis.yml and
.github/workflows/publish-docs-to-s3.yml
+ - name: Setup Java
+ uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654 #
v5.2.0
+ with:
+ distribution: 'temurin'
+ java-version: '11'
Review Comment:
The value is only used once in this file, I opted to not use `env` so the
comment can be kept close to the usage instead of toward the top. (Same as
codeql-analysis.yml)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
