Skip dry runs in PRs based on diff (#10)

Skip workflow dry run jobs in PRs based on whether diff contains files
relevant to that workflow.

Author: gabrielfeo

.github/workflows/check-new-api-spec.yml

@@ -10,14 +10,16 @@ on:
         type: boolean
         required: false
   workflow_call:
+    inputs:
+      dry_run:
+        description: 'Dry run'
+        type: boolean
+        required: true
 
 defaults:
   run:
     shell: bash
 
-env:
-  DRY_RUN: ${{ github.event.inputs.dry_run || github.event_name == 'pull_request' }}
-
 jobs:
 
   check-new-api-spec:
@@ -33,7 +35,7 @@ jobs:
           [[ -n "$version" ]]
           echo "SPEC_VERSION=$version" | tee -a "$GITHUB_ENV"
       - name: 'Create PR'
-        if: ${{ env.DRY_RUN == 'false' }}
+        if: ${{ !inputs.dry_run }}
         uses: peter-evans/create-pull-request@v4
         with:
           branch: "feature/api-spec-${{ env.SPEC_VERSION }}"

.github/workflows/pr.yml

@@ -45,8 +45,43 @@ jobs:
         if: ${{ steps.diff.outputs.all_modified_files }}
         run: python3 -m unittest discover -bs .github/scripts
 
-  publish-javadoc-dry-run:
+  choose-dry-runs:
+    runs-on: ubuntu-latest
+    outputs:
+      dry-run-publish-javadoc: ${{ steps.diff-publish-javadoc.outputs.all_modified_files }}
+      dry-run-check-new-api-spec: ${{ steps.diff-check-new-api-spec.outputs.all_modified_files }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+      - name: Diff publish-javadoc.yml
+        id: diff-publish-javadoc
+        uses: tj-actions/changed-files@v24
+        with:
+          files: |
+            .github/workflows/publish-javadoc.yml
+            .github/scripts/*
+            **/src/**
+            **/*.gradle*
+            ./gradle/*
+      - name: Diff check-new-api-spec.yml
+        id: diff-check-new-api-spec
+        uses: tj-actions/changed-files@v24
+        with:
+          files: |
+            .github/workflows/check-new-api-spec.yml
+            .github/scripts/*
+            gradle.properties
+
+  dry-run-publish-javadoc:
+    needs: [choose-dry-runs]
+    if: ${{ needs.choose-dry-runs.outputs.dry-run-publish-javadoc }}
     uses: ./.github/workflows/publish-javadoc.yml
+    with:
+      dry_run: true
 
-  check-new-api-spec-dry-run:
+  dry-run-check-new-api-spec:
+    needs: [choose-dry-runs]
+    if: ${{ needs.choose-dry-runs.outputs.dry-run-check-new-api-spec }}
     uses: ./.github/workflows/check-new-api-spec.yml
+    with:
+      dry_run: true

.github/workflows/publish-javadoc.yml

@@ -10,13 +10,17 @@ on:
         type: boolean
         required: false
   workflow_call:
+    inputs:
+      dry_run:
+        description: 'Dry run'
+        type: boolean
+        required: true
 
 defaults:
   run:
     shell: bash
 
 env:
-  DRY_RUN: ${{ github.event.inputs.dry_run || github.event_name == 'pull_request' }}
   JAR_NAME: gradle-enterprise-api-kotlin-SNAPSHOT-javadoc.jar
 
 jobs:
@@ -39,21 +43,24 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v3
-      - name: Fetch branch
-        run: git fetch origin gh-pages && git checkout FETCH_HEAD
+        with:
+          ref: gh-pages
       - name: Download javadoc
         uses: actions/download-artifact@v3
       - name: Unzip javadoc
-        run: rm -rf docs && unzip "javadoc/$JAR_NAME" -d docs
+        run: |
+          rm -rf docs
+          unzip "javadoc/$JAR_NAME" -d docs
+          rm -rf javadoc
       - name: Commit
         run: |
           git config user.name github-actions
           git config user.email github-actions@github.com
           git add docs
-          git commit -m "Add ${{ github.ref_name }} javadoc"
+          git commit --allow-empty -m "Add ${{ github.ref_name }} javadoc"
       - name: Push
         run: |
-          if [[ "$DRY_RUN" == 'true' ]]; then
+          if [[ "${{ inputs.dry_run }}" == 'true' ]]; then
             args=' --dry-run'
           fi
-          git push origin HEAD:gh-pages $args
+          git push $args