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

mchades pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new ff4df29f1  [#4002] Improvement(gradle): remove skipPyClientITs 
parameter in Gradle (#4125)
ff4df29f1 is described below

commit ff4df29f1c42ef6c9e42060699afb73937e57f3b
Author: Weison Wei <[email protected]>
AuthorDate: Mon Jul 15 11:15:49 2024 +0800

     [#4002] Improvement(gradle): remove skipPyClientITs parameter in Gradle 
(#4125)
    
    ### What changes were proposed in this pull request?
    
    remove duplicated  `skipPyClientITs` parameter in Gradle
    
    ### Why are the changes needed?
    
    Fix: #4002
    
    ### Does this PR introduce _any_ user-facing change?
    
    N/A
    
    ### How was this patch tested?
    set `skipITs=false` to run integration test
---
 clients/client-python/build.gradle.kts | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/clients/client-python/build.gradle.kts 
b/clients/client-python/build.gradle.kts
index 2cf83c376..044cdb061 100644
--- a/clients/client-python/build.gradle.kts
+++ b/clients/client-python/build.gradle.kts
@@ -249,15 +249,14 @@ tasks {
 
   val test by registering(VenvTask::class) {
     val skipUTs = project.hasProperty("skipTests")
-    val skipPyClientITs = project.hasProperty("skipPyClientITs")
     val skipITs = project.hasProperty("skipITs")
-    val skipAllTests = skipUTs && (skipITs || skipPyClientITs)
+    val skipAllTests = skipUTs && skipITs
     if (!skipAllTests) {
       dependsOn(pipInstall, pylint)
       if (!skipUTs) {
         dependsOn(unitTests)
       }
-      if (!skipITs && !skipPyClientITs) {
+      if (!skipITs) {
         dependsOn(integrationTest)
       }
     }

Reply via email to