[llvm] [Github] Fetch all commits in PR for code formatting checks (PR #69766)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 29 00:15:50 PDT 2023


boomanaiden154 wrote:

> Nicely done! It looks like a good approach. If I understand the spare checkout correctly it's only to get the scripts to run. Can't we fetch these files with curl instead of a sparse checkout? Should be faster and it's what we do in other places.

Yes, the sparse checkout is only to get the scripts that need to run. I don't think the time difference is significant. The sparse checkout takes about two seconds from my testing and it has a couple nice properties like using the scripts from the target branch rather than an arbitrary ref (probably `main`) in the curl invocations which makes future testing/adaptation quite a bit easier.

https://github.com/llvm/llvm-project/pull/69766


More information about the llvm-commits mailing list