Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add sccache and use nextest in our workflow #265

Merged
merged 6 commits into from
Jul 30, 2024
Merged

Conversation

ggonzalez94
Copy link
Collaborator

@ggonzalez94 ggonzalez94 commented Jul 26, 2024

Fixes #259

Our workflows take about 50min each on every PR(on main it is longer given we also run it on mac OS). This can be improved using sccache which avoids compilation when possible. It is common in polkadot projects to have this given long build times.
With this, subsequent runs take around 35min.

  • Add sccache step
  • Use nextest for tests, which is up to 3x faster than cargo test, and also shows the results in a much nicer format. It also shows you if you have some slow tests. Currently this is not a big advantage, since our tests are quick, but even the formatting is worth it I think.

@ggonzalez94 ggonzalez94 requested review from ozgunozerk and 4meta5 July 26, 2024 13:50
Copy link

netlify bot commented Jul 26, 2024

Deploy Preview for docs-oz-polkadot canceled.

Name Link
🔨 Latest commit 5a5e059
🔍 Latest deploy log https://app.netlify.com/sites/docs-oz-polkadot/deploys/66a931c887382a0008c6d3b7

Copy link
Contributor

@4meta5 4meta5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great DX improvement for all of us!

@ggonzalez94 ggonzalez94 marked this pull request as ready for review July 26, 2024 17:58
Copy link
Collaborator

@ozgunozerk ozgunozerk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot! Seems like great improvements ⚡

@ggonzalez94 ggonzalez94 merged commit d14851c into main Jul 30, 2024
6 checks passed
@ggonzalez94 ggonzalez94 deleted the wofklow-cache branch July 30, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Implement sccache and cargo registry & git sources cache in the workflows
3 participants