Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Speedup CI#9970

Merged
kyri-petrou merged 3 commits intozio:series/2.xfrom
kyri-petrou:try-fetch-depth
Jun 14, 2025
Merged

Speedup CI#9970
kyri-petrou merged 3 commits intozio:series/2.xfrom
kyri-petrou:try-fetch-depth

Conversation

@kyri-petrou
Copy link
Contributor

@kyri-petrou kyri-petrou commented Jun 13, 2025

Some time in the repo's past, very large files were being committed, and fetching the full history takes a long time (~2 and a half minutes in CI). Unfortunately, for some steps we have to get the history because we need to resolve the latest tag. Luckily, we can do that by using a relatively large fetch-depth, but not large enough that'll go all the way back to the start of the repo's history.

Hopefully this will work; every time I tried to "solve" this it resulted in me breaking the release job. I think this time it'll work though (famous last words)

@kyri-petrou kyri-petrou changed the title Try settings fetch-depth in CI Try defining fetch-depth in CI Jun 13, 2025
@kyri-petrou kyri-petrou changed the title Try defining fetch-depth in CI Speedup CI Jun 13, 2025
@kyri-petrou kyri-petrou marked this pull request as ready for review June 13, 2025 16:02
@kyri-petrou kyri-petrou enabled auto-merge (squash) June 13, 2025 16:02
@kyri-petrou kyri-petrou requested a review from hearnadam June 13, 2025 16:02
hearnadam
hearnadam previously approved these changes Jun 13, 2025
@plokhotnyuk
Copy link
Contributor

How about running on ubuntu-latest-arm for even greater boost of CI jobs?

@kyri-petrou kyri-petrou disabled auto-merge June 14, 2025 13:44
@kyri-petrou kyri-petrou merged commit f1016e5 into zio:series/2.x Jun 14, 2025
19 checks passed
@kyri-petrou kyri-petrou deleted the try-fetch-depth branch June 14, 2025 13:44
@kyri-petrou
Copy link
Contributor Author

How about running on ubuntu-latest-arm for even greater boost of CI jobs?

@plokhotnyuk Thanks for the recommendation, I'll give that a go in a separate PR. I'm a bit worried that it might cause issues with the Scala Native tests but we'll see 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants