appveyor / ci

AppVeyor community support repository
https://www.appveyor.com
344 stars 64 forks source link

missing `depth` parameter #3876

Open ttodua opened 1 year ago

ttodua commented 1 year ago

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.

FeodorFitsner commented 1 year ago

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 1 year ago

thanks! plz let us know whenever it will be implemented

ttodua commented 1 year ago

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

ttodua commented 5 months ago

@FeodorFitsner any thoughts whether this can be fixed? if you look into https://ci.appveyor.com/project/ccxt/ccxt/history you will see that 3GB of repo is being fetched on each commit. it is huge performance penalty, so you might fix that problem, i think other repos will be bleeding from that too.

the similar issue was on travis, and after i had reported, they had fixed it a while.