Copilot commented on code in PR #2789:
URL: 
https://github.com/apache/incubator-hugegraph/pull/2789#discussion_r2137131055


##########
.github/scripts/prepare-backend.sh:
##########
@@ -0,0 +1,13 @@
+#!/bin/bash
+set -e
+
+BACKEND=$1
+
+echo "Preparing backend ci environment for $BACKEND..."
+
+if [[ "$BACKEND" == "hstore" ]]; then
+  mvn package -U -Dmaven.javadoc.skip=true -Dmaven.test.skip=true -ntp

Review Comment:
   [nitpick] Running a full `mvn package` can be time-consuming. If you only 
need dependencies or compilation artifacts for HStore, consider using `mvn 
dependency:go-offline` or `mvn compile` to speed up the CI run.
   ```suggestion
     mvn dependency:go-offline -U -ntp
     mvn compile -Dmaven.javadoc.skip=true -Dmaven.test.skip=true -ntp
   ```



##########
.github/workflows/server-ci.yml:
##########
@@ -100,3 +100,5 @@ jobs:
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           file: ${{ env.REPORT_DIR }}/*.xml
+
+

Review Comment:
   There are two extra blank lines at the end of this job—removing them will 
clean up the YAML and prevent formatting noise.
   ```suggestion
   
   ```



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to