You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by xu...@apache.org on 2023/10/31 10:35:02 UTC

(incubator-opendal) branch fix-python-ci updated: Fix plan

This is an automated email from the ASF dual-hosted git repository.

xuanwo pushed a commit to branch fix-python-ci
in repository https://gitbox.apache.org/repos/asf/incubator-opendal.git


The following commit(s) were added to refs/heads/fix-python-ci by this push:
     new e0997945d Fix plan
e0997945d is described below

commit e0997945d031190a805f500fb9eba80655b49f44
Author: Xuanwo <gi...@xuanwo.io>
AuthorDate: Tue Oct 31 18:34:54 2023 +0800

    Fix plan
    
    Signed-off-by: Xuanwo <gi...@xuanwo.io>
---
 .github/scripts/behavior_test/plan.py | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/.github/scripts/behavior_test/plan.py b/.github/scripts/behavior_test/plan.py
index 5cea9c2bd..e734a3d88 100755
--- a/.github/scripts/behavior_test/plan.py
+++ b/.github/scripts/behavior_test/plan.py
@@ -213,6 +213,12 @@ def generate_binding_python_cases(
     cases: list[dict[str, str]], hint: Hint
 ) -> list[dict[str, str]]:
     cases = unique_cases(cases)
+
+    # REMOVE ME after https://github.com/apache/incubator-opendal/issues/3429 addressed.
+    #
+    # Sftp can't pass on python bindings, remove it for now.
+    cases = [v for v in cases if v["service"] != "sftp"]
+
     if os.getenv("GITHUB_IS_PUSH") == "true":
         return cases
 
@@ -226,11 +232,6 @@ def generate_binding_python_cases(
 
     # Filter all cases that not shown un in changed files
     cases = [v for v in cases if v["service"] in hint.services]
-
-    # REMOVE ME after https://github.com/apache/incubator-opendal/issues/3429 addressed.
-    #
-    # Sftp can't pass on python bindings, remove it for now.
-    cases = [v for v in cases if v["service"] != "sftp"]
     return cases