Merge pull request #8066 from jrjohnson/fix-automerge-again-ugh

Fix Variables in Workflow
ilios · Aug 14, 2024 · eef95af · eef95af
2 parents 8afe111 + 5419f42
commit eef95af
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/.github/workflows/dependabot-automerge.yml b/.github/workflows/dependabot-automerge.yml
@@ -17,12 +17,16 @@ jobs:
           github-token: "${{ secrets.GITHUB_TOKEN }}"
       - run: >
           npx in-string-list ${{ steps.metadata.outputs.dependency-names }} ${{ env.SAFE_DEPENDENCIES }}
-          && echo "isSafe=yes" >> "$GITHUB_OUTPUT"
+          && echo "isSafe=yes" >> "$GITHUB_ENV"
       - run: >
           test ${{ steps.metadata.outputs.update-type }} != 'version-update:semver-major'
-          && echo "isNotMajor=yes" >> "$GITHUB_OUTPUT"
+          && echo "isNotMajor=yes" >> "$GITHUB_ENV"
+      - run: |
+          echo ${{ env.isSafe }}
+          echo ${{ env.isNotMajor }}
+          echo (test -z ${{env.isSafe}} && test -z ${{env.isNotMajor}})
       - id: should_merge
-        run: (test -z {{env.isSafe}} && test -z {{env.isNotMajor}}) && echo "shouldMerge=yes" >> "$GITHUB_OUTPUT"
+        run: (test -z ${{env.isSafe}} && test -z ${{env.isNotMajor}}) && echo "shouldMerge=yes" >> "$GITHUB_OUTPUT"
   auto_merge:
     runs-on: ubuntu-latest
     needs: check