ci: run on pull requests only

synapsecns · Sep 25, 2024 · 0a01d60 · 0a01d60
1 parent 832f973
commit 0a01d60
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/.github/workflows/solidity.yml b/.github/workflows/solidity.yml
@@ -243,7 +243,8 @@ jobs:
   gas-diff:
     runs-on: ubuntu-latest
     name: Foundry Gas Diff
-    if: ${{ needs.changes.outputs.package_count > 0 }}
+    # This action needs to be triggered on Pull Requests only, assuming there are changes to the package in question.
+    if: ${{ (github.event_name == 'pull_request' || github.event_name == 'pull_request_target') && needs.changes.outputs.package_count > 0 }}
     needs: changes
     strategy:
       fail-fast: false
@@ -288,7 +289,7 @@ jobs:
         id: gas_diff
 
       - name: Add gas diff to sticky comment
-        if: github.event_name == 'pull_request' || github.event_name == 'pull_request_target'
+        if: ${{ github.event_name == 'pull_request' || github.event_name == 'pull_request_target' }}
         uses: marocchino/sticky-pull-request-comment@v2
         with:
           # delete the comment in case changes no longer impact gas costs