From cf3c205fa53c9f309a7c3315fed76a7a80c22c97 Mon Sep 17 00:00:00 2001
From: Zhiwei <43905414+ChihweiLHBird@users.noreply.github.com>
Date: Sun, 5 Dec 2021 15:05:07 -0700
Subject: [PATCH] Fixed: CI Runs on Ready PRs Only (#2324)

* Revert "Update tests to not run all the time (#2311)"

This reverts commit 2c03eee32948aa7d0c2f156244c25e22541f9e21.

* Make CI only runs on PRs that are ready

* Remove CI Tasks on Push
---
 .github/workflows/codeql-analysis.yml | 3 ++-
 .github/workflows/coverage.yml        | 3 ++-
 .github/workflows/pr-bandit.yml       | 3 ++-
 .github/workflows/pr-docs.yml         | 3 ++-
 .github/workflows/pr-linter.yml       | 3 ++-
 .github/workflows/pr-python310.yml    | 5 +++--
 .github/workflows/pr-python37.yml     | 3 ++-
 .github/workflows/pr-python38.yml     | 3 ++-
 .github/workflows/pr-python39.yml     | 3 ++-
 .github/workflows/pr-type-check.yml   | 3 ++-
 .github/workflows/pr-windows.yml      | 3 ++-
 11 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
index 6f379006..5108c247 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -5,12 +5,13 @@ on:
     branches: [ main ]
   pull_request:
     branches: [ main ]
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
   schedule:
     - cron: '25 16 * * 0'
 
 jobs:
   analyze:
+    if: github.event.pull_request.draft == false
     name: Analyze
     runs-on: ubuntu-latest
 
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
index 4293ae74..c478a961 100644
--- a/.github/workflows/coverage.yml
+++ b/.github/workflows/coverage.yml
@@ -11,9 +11,10 @@ on:
   pull_request:
     paths:
       - "!*.MD"
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 jobs:
   test:
+    if: github.event.pull_request.draft == false
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
diff --git a/.github/workflows/pr-bandit.yml b/.github/workflows/pr-bandit.yml
index 97aade87..ca91312a 100644
--- a/.github/workflows/pr-bandit.yml
+++ b/.github/workflows/pr-bandit.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   bandit:
+    if: github.event.pull_request.draft == false
     name: type-check-${{ matrix.config.python-version }}
     runs-on: ${{ matrix.os }}
     strategy:
diff --git a/.github/workflows/pr-docs.yml b/.github/workflows/pr-docs.yml
index 2d01315a..7b3c2f6e 100644
--- a/.github/workflows/pr-docs.yml
+++ b/.github/workflows/pr-docs.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   docsLinter:
+    if: github.event.pull_request.draft == false
     name: Lint Documentation
     runs-on: ubuntu-latest
     strategy:
diff --git a/.github/workflows/pr-linter.yml b/.github/workflows/pr-linter.yml
index 83f7e458..9ed45d0a 100644
--- a/.github/workflows/pr-linter.yml
+++ b/.github/workflows/pr-linter.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   linter:
+    if: github.event.pull_request.draft == false
     name: lint
     runs-on: ${{ matrix.os }}
     strategy:
diff --git a/.github/workflows/pr-python310.yml b/.github/workflows/pr-python310.yml
index 327672a9..f3f7c607 100644
--- a/.github/workflows/pr-python310.yml
+++ b/.github/workflows/pr-python310.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
-  testPy39:
+  testPy310:
+    if: github.event.pull_request.draft == false
     name: ut-${{ matrix.config.tox-env }}-${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
diff --git a/.github/workflows/pr-python37.yml b/.github/workflows/pr-python37.yml
index 3fb65117..50f79c6e 100644
--- a/.github/workflows/pr-python37.yml
+++ b/.github/workflows/pr-python37.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   testPy37:
+    if: github.event.pull_request.draft == false
     name: ut-${{ matrix.config.tox-env }}-${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
diff --git a/.github/workflows/pr-python38.yml b/.github/workflows/pr-python38.yml
index f8bcb062..1e0b8050 100644
--- a/.github/workflows/pr-python38.yml
+++ b/.github/workflows/pr-python38.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   testPy38:
+    if: github.event.pull_request.draft == false
     name: ut-${{ matrix.config.tox-env }}-${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
diff --git a/.github/workflows/pr-python39.yml b/.github/workflows/pr-python39.yml
index 3e1967bd..1abd6bcb 100644
--- a/.github/workflows/pr-python39.yml
+++ b/.github/workflows/pr-python39.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   testPy39:
+    if: github.event.pull_request.draft == false
     name: ut-${{ matrix.config.tox-env }}-${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
diff --git a/.github/workflows/pr-type-check.yml b/.github/workflows/pr-type-check.yml
index 73cfba90..2fae03be 100644
--- a/.github/workflows/pr-type-check.yml
+++ b/.github/workflows/pr-type-check.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   typeChecking:
+    if: github.event.pull_request.draft == false
     name: type-check-${{ matrix.config.python-version }}
     runs-on: ${{ matrix.os }}
     strategy:
diff --git a/.github/workflows/pr-windows.yml b/.github/workflows/pr-windows.yml
index 40ecb020..9721b5b5 100644
--- a/.github/workflows/pr-windows.yml
+++ b/.github/workflows/pr-windows.yml
@@ -3,10 +3,11 @@ on:
   pull_request:
     branches:
       - main
-    types: [review_requested, ready_for_review]
+    types: [opened, synchronize, reopened, ready_for_review]
 
 jobs:
   testsOnWindows:
+    if: github.event.pull_request.draft == false
     name: ut-${{ matrix.config.tox-env }}
     runs-on: windows-latest
     strategy: