appveyor/ci

missing `depth` parameter

ttodua opened this issue · 4 comments

ttodua commented

in appveyor.yml you can set clone_depth:1 however the fetch that is being made :

https://ci.appveyor.com/project/ccxt/ccxt/builds/47088338#L3

does not respect depth:1 and it fetches full ref, which is around 3gb for that specific repo in my example link. The same issue was happening in travis, and we reported them and they have fixed (now they use depth param, i.e. https://app.travis-ci.com/github/ccxt/ccxt/builds/263174800#L173 ).

So, please add depth param in appveyor too.

Sure, in the meantime you can use a custom clone script:

clone_script:
- ps: |
    if(-not $env:appveyor_pull_request_number) {
        git clone -q --branch=$env:appveyor_repo_branch https://github.com/$env:appveyor_repo_name.git $env:appveyor_build_folder
        git checkout -qf $env:appveyor_repo_commit
    } else {
        git clone -q https://github.com/$env:appveyor_repo_name.git $env:appveyor_build_folder
        git fetch -q origin +refs/pull/$env:appveyor_pull_request_number/merge:
        git checkout -qf FETCH_HEAD
    }
ttodua commented

thanks! plz let us know whenever it will be implemented

ttodua commented

any chance to add depth param like it's here on 173th line? https://app.travis-ci.com/github/ccxt/ccxt/builds/263174800#L173