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_r384925420
##########
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]:"
+ auth_header="$(git config --local --get
http.https://github.com/.extraheader)"
Review comment:
> Don't copy the thing if that is not necessary.
get it.
> Why should be different? I am not following. We are using the same
parameter to download from GitHub too.
>
> Could you explain your meaning of SHOULD?
It is indeed that I understand it wrong, `auth_header` can be deleted for sw
project, because english is not good, the description is not accurate, it is
possible.
----------------------------------------------------------------
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
