FANNG1 commented on code in PR #10385:
URL: https://github.com/apache/gravitino/pull/10385#discussion_r2957559610


##########
.github/workflows/build.yml:
##########
@@ -84,7 +84,7 @@ jobs:
 
       - name: Build with Gradle
         run: |
-          ./gradlew release -x test
+          ./gradlew assemble

Review Comment:
   Agreed that `build -x test` would run broader checks such as 
Spotless/license verification. In this workflow, though, those checks are 
already covered by the main `build` job, which still runs Gradle `build`. The 
`compile-check` job is intended to stay as a lightweight fallback path, so I 
prefer to keep `assemble` here to avoid pulling extra non-Java subproject 
`build` paths into that fallback job.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to