aderm commented on a change in pull request #4419: [WIP]Update to 
actions/checkout@v2, fix checks error: reference is not a tree.
URL: https://github.com/apache/skywalking/pull/4419#discussion_r384827568
 
 

 ##########
 File path: .github/workflows/ci-it.yaml
 ##########
 @@ -32,9 +32,15 @@ jobs:
     strategy:
       fail-fast: true
     steps:
-      - uses: actions/checkout@v1
-        with:
-          submodules: true
+      - uses: actions/checkout@v2
+      - name: Checkout submodules
+        shell: bash
+        run: |
+          # If your submodules are configured to use SSH instead of HTTPS 
please uncomment the following line
+          # git config --global url."https://github.com/".insteadOf 
"[email protected]:"
 
 Review comment:
   > `v2` is not compatible with `v1`, at least the parameters are not 
compatible, and I suggest you fix this in the pull request where you encounter 
the issue, which can verify the fix works
   
   Sorry for not being clearly marked. It is true that the current V2 does not 
support the submodule function. Later, try to complete the verification in the 
local environment before submitting the PR.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to