Compute check_code_coverage once and pass as variable

Author: ghpvnist

.github/workflows/_build_plugin.yml

@@ -16,21 +16,17 @@ on:
         type: number
         description: Timeout in minutes for the build job
         default: 120
+      has_code_changes:
+        required: false
+        type: boolean
+        description: Whether to run full workflow or not
+        default: true
     secrets:
       gcloud-service-key:
         required: true
         description: Secret to access Bazel build cache
 jobs:
-  check_code_changes:
-    name: Check Code Changes
-    uses: ./.github/workflows/_check_code_changes.yml
-    with:
-      event_name: ${{ github.event_name }}
-      # For pull_request, use PR's base and head. For push, use event's before and sha.
-      base_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.base.sha || github.event.before }}
-      head_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
   build:
-    needs: [check_code_changes]
     runs-on: ${{ inputs.runner }}
     timeout-minutes: ${{ inputs.timeout-minutes }}
     container:
@@ -41,31 +37,31 @@ jobs:
       BAZEL_REMOTE_CACHE: ${{ github.event_name == 'push' || github.event.pull_request.head.repo.full_name == github.repository }}
     steps:
       - name: Checkout actions
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: actions/checkout@v4
         with:
           sparse-checkout: |
             .github/workflows/setup
           path: .actions
       - name: Setup
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: ./.actions/.github/workflows/setup
         with:
           torch-commit: ${{ inputs.torch-commit }}
           cuda: true
       - name: Build
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         shell: bash
         run: |
           cd pytorch/xla/infra/ansible
           ansible-playbook playbook.yaml -vvv -e "stage=build_plugin arch=amd64 accelerator=cuda cuda_compute_capabilities=5.2,7.5,8.6 src_root=${GITHUB_WORKSPACE} cache_suffix=-ci" --skip-tags=fetch_srcs,install_deps
       - name: Upload wheel
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: actions/upload-artifact@v4
         with:
           name: cuda-plugin
           path: /dist/*.whl
       - name: Report no code changes
-        if: needs.check_code_changes.outputs.has_code_changes == 'false'
+        if: ${{ !inputs.has_code_changes }}
         run: |
           echo "No code changes were detected that require running the full test suite."

.github/workflows/_build_torch_with_cuda.yml

@@ -20,17 +20,13 @@ on:
         type: number
         description: Timeout in minutes for the build job
         default: 120
+      has_code_changes:
+        required: false
+        type: boolean
+        description: Whether to run full workflow or not
+        default: true
 jobs:
-  check_code_changes:
-    name: Check Code Changes
-    uses: ./.github/workflows/_check_code_changes.yml
-    with:
-      event_name: ${{ github.event_name }}
-      # For pull_request, use PR's base and head. For push, use event's before and sha.
-      base_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.base.sha || github.event.before }}
-      head_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
   build:
-    needs: [check_code_changes]
     runs-on: ${{ inputs.runner }}
     timeout-minutes: ${{ inputs.timeout-minutes }}
     container:
@@ -42,31 +38,31 @@ jobs:
       MAX_JOBS: 24
     steps:
       - name: Checkout actions
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: actions/checkout@v4
         with:
           sparse-checkout: |
             .github/workflows/setup
           path: .actions
       - name: Setup
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: ./.actions/.github/workflows/setup
         with:
           torch-commit: ${{ inputs.torch-commit }}
           cuda: true
       - name: Build PyTorch with CUDA enabled
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         shell: bash
         run: |
           cd pytorch
           python setup.py bdist_wheel
       - name: Upload wheel
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: actions/upload-artifact@v4
         with:
           name: torch-with-cuda
           path: pytorch/dist/*.whl
       - name: Report no code changes
-        if: needs.check_code_changes.outputs.has_code_changes == 'false'
+        if: ${{ !inputs.has_code_changes }}
         run: |
           echo "No code changes were detected that require running the full test suite."

.github/workflows/_build_torch_xla.yml

@@ -20,21 +20,17 @@ on:
         type: number
         description: Timeout in minutes for the build job
         default: 120
+      has_code_changes:
+        required: false
+        type: boolean
+        description: Whether to run full workflow or not
+        default: true
     secrets:
       gcloud-service-key:
         required: true
         description: Secret to access Bazel build cache
 jobs:
-  check_code_changes:
-    name: Check Code Changes
-    uses: ./.github/workflows/_check_code_changes.yml
-    with:
-      event_name: ${{ github.event_name }}
-      # For pull_request, use PR's base and head. For push, use event's before and sha.
-      base_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.base.sha || github.event.before }}
-      head_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
   build:
-    needs: [check_code_changes]
     runs-on: ${{ inputs.runner }}
     timeout-minutes: ${{ inputs.timeout-minutes }}
     container:
@@ -48,37 +44,37 @@ jobs:
       # Need to check out local composite actions before using them
       # https://github.com/orgs/community/discussions/11771
       - name: Checkout actions
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: actions/checkout@v4
         with:
           sparse-checkout: |
             .github/workflows/setup
           path: .actions
       - name: Setup
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: ./.actions/.github/workflows/setup
         with:
           torch-commit: ${{ inputs.torch-commit }}
       - name: Build
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         shell: bash
         run: |
           cd pytorch/xla/infra/ansible
           ansible-playbook playbook.yaml -vvv -e "stage=build arch=amd64 accelerator=tpu src_root=${GITHUB_WORKSPACE} bundle_libtpu=0 build_cpp_tests=1 git_versioned_xla_build=1 cache_suffix=-ci" --skip-tags=fetch_srcs,install_deps
       - name: Upload wheel
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: actions/upload-artifact@v4
         with:
           name: torch-xla-wheels
           path: /dist/*.whl
       - name: Upload CPP test binaries
-        if: needs.check_code_changes.outputs.has_code_changes == 'true'
+        if: ${{ inputs.has_code_changes }}
         uses: actions/upload-artifact@v4
         with:
           name: cpp-test-bin
           path: /tmp/test/bin
       - name: Report no code changes
-        if: needs.check_code_changes.outputs.has_code_changes == 'false'
+        if: ${{ !inputs.has_code_changes }}
         run: |
           echo "No code changes were detected that require running the full test suite."
 

.github/workflows/_docs.yml

@@ -11,20 +11,16 @@ on:
         type: string
         description: Runner type for the test
         default: linux.4xlarge
+      has_code_changes:
+        required: false
+        type: boolean
+        description: Whether to run full workflow or not
+        default: true
     secrets:
       torchxla-bot-token:
         required: true
 jobs:
-  check_code_changes:
-    name: Check Code Changes
-    uses: ./.github/workflows/_check_code_changes.yml
-    with:
-      event_name: ${{ github.event_name }}
-      # For pull_request, use PR's base and head. For push, use event's before and sha.
-      base_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.base.sha || github.event.before }}
-      head_sha: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
   build-docs:
-    needs: [check_code_changes]
     runs-on: ubuntu-24.04
     timeout-minutes: 45
     container:
@@ -33,52 +29,50 @@ jobs:
       BRANCH_NAME: ${{ github.ref_name }}
     steps:
     - name: Fetch wheels
-      if: needs.check_code_changes.outputs.has_code_changes == 'true'
+      if: ${{ inputs.has_code_changes }}
       uses: actions/download-artifact@v4
       with:
         name: torch-xla-wheels
         path: /tmp/wheels/
     - name: Install wheels
-      if: needs.check_code_changes.outputs.has_code_changes == 'true'
+      if: ${{ inputs.has_code_changes }}
       shell: bash
       run: |
         pip install /tmp/wheels/*.whl
     - name: Checkout PyTorch/XLA Repo
-      if: needs.check_code_changes.outputs.has_code_changes == 'true'
+      if: ${{ inputs.has_code_changes }}
       uses: actions/checkout@v4
       with:
         path: pytorch/xla
     - name: Build docs
-      if: needs.check_code_changes.outputs.has_code_changes == 'true'
+      if: ${{ inputs.has_code_changes }}
       shell: bash
       run: |
         cd pytorch/xla/docs
         pip install -r requirements.txt
         sphinx-build -b html source build
     - name: Checkout GitHub Pages
-      if: needs.check_code_changes.outputs.has_code_changes == 'true'
+      if: ${{ inputs.has_code_changes }}
       uses: actions/checkout@v4
       with:
         path: gh-pages
         ref: gh-pages
         token: ${{ github.event_name == 'push' && secrets.torchxla-bot-token || github.token }}
     - name: Merge changes
-      if: needs.check_code_changes.outputs.has_code_changes == 'true'
+      if: ${{ inputs.has_code_changes }}
       shell: bash
       run: |
         subdir=${{ env.BRANCH_NAME == 'master' && 'master' || format('{0}/{1}', 'release', env.BRANCH_NAME) }}
         mkdir -p gh-pages/$subdir
         cp -fR pytorch/xla/docs/build/* gh-pages/$subdir
     - name: Upload preview as artifact
-      if: needs.check_code_changes.outputs.has_code_changes == 'true'
+      if: ${{ inputs.has_code_changes }}
       uses: actions/upload-artifact@v4
       with:
         name: github-pages
         path: pytorch/xla/docs/build/
     - name: Deploy
-      if: >
-        needs.check_code_changes.outputs.has_code_changes == 'true' &&
-        github.event_name == 'push'
+      if: ${{ inputs.has_code_changes }} && github.event_name == 'push'
       shell: bash
       run: |
         cd gh-pages
@@ -88,6 +82,6 @@ jobs:
         git diff --cached --exit-code || git commit -m "Update doc from commit ${{ github.sha }}"
         git push origin gh-pages
     - name: Report no code changes
-      if: needs.check_code_changes.outputs.has_code_changes == 'false'
+      if: ${{ !inputs.has_code_changes }}
       run: |
         echo "No code changes were detected that require running the full test suite."