Not triggering unit tests on Draft PR (#2033)

* Not triggering unit tests on Draft PR Signed-off-by: Jun Ki Min <[email protected]> * Change a PR-triggering file to test Signed-off-by: Jun Ki Min <[email protected]> --------- Signed-off-by: Jun Ki Min <[email protected]>
recommenders-team · Nov 3, 2023 · 4ee5262 · 4ee5262
1 parent 2c6b258
commit 4ee5262
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.github/workflows/azureml-unit-tests.yml b/.github/workflows/azureml-unit-tests.yml
@@ -19,6 +19,7 @@ on:
       - tests/**
       - '!tests/**/*.md'
       - setup.py
+    types: [opened, synchronize, reopened, ready_for_review]
 
   # Enable manual trigger
   workflow_dispatch:
@@ -46,6 +47,7 @@ jobs:
         shell: bash
     outputs:
       test_groups: ${{ steps.get_test_groups.outputs.test_groups }}
+    if: github.event.pull_request.draft == false
 
   execute-tests:
     needs: get-test-groups
@@ -69,3 +71,4 @@ jobs:
           AZUREML_TEST_SUBID: ${{ secrets.AZUREML_TEST_SUBID }}
           PYTHON_VERSION: ${{ matrix.python-version }}
           TEST_GROUP: ${{ matrix.test-group }}
+    if: github.event.pull_request.draft == false
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -75,7 +75,6 @@ def spark(tmp_path_factory, app_name="Sample", url="local[*]"):
     Returns:
         SparkSession: new Spark session
     """
-
     with TemporaryDirectory(dir=tmp_path_factory.getbasetemp()) as td:
         config = {
             "spark.local.dir": td,