[ci] fix some flow would still be allowed to run after being marked a…

…s draft
chipsalliance · Dec 5, 2024 · 23bcbe5 · 23bcbe5
1 parent 3336e85
commit 23bcbe5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/t1rocket.yml b/.github/workflows/t1rocket.yml
@@ -87,9 +87,10 @@ jobs:
   report:
     name: "Report T1Rocket VCS CI result"
     # Don't run report when:
+    #   - marked draft ( we don't need to run report for draft PR )
     #   - user cancel ( we don't need report at this case )
     #   - PR from outside repository ( we don't have permission to push commit into fork repository )
-    if: ${{ !cancelled() && github.event.pull_request.head.repo.full_name == github.repository }}
+    if: ${{ !cancelled() && github.event.pull_request.head.repo.full_name == github.repository && !github.event.pull_request.draft }}
     needs: [run-testcases]
     runs-on: [self-hosted, linux, nixos]
     permissions:

diff --git a/.github/workflows/vcs.yml b/.github/workflows/vcs.yml
@@ -87,9 +87,10 @@ jobs:
   report:
     name: "Report VCS CI result"
     # Don't run report when:
+    #   - marked draft ( we don't need to run report for draft PR )
     #   - user cancel ( we don't need report at this case )
     #   - PR from outside repository ( we don't have permission to push commit into fork repository )
-    if: ${{ !cancelled() && github.event.pull_request.head.repo.full_name == github.repository }}
+    if: ${{ !cancelled() && github.event.pull_request.head.repo.full_name == github.repository && !github.event.pull_request.draft }}
     needs: [run-testcases]
     runs-on: [self-hosted, linux, nixos]
     permissions: