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

chore(ci): update setup-rust cache strategy #2166

Merged
merged 1 commit into from
Mar 22, 2024

Conversation

Sec-ant
Copy link
Contributor

@Sec-ant Sec-ant commented Mar 22, 2024

Summary

Our CI is encountering this error: Failed to save: Cache service responded with 429 during upload chunk., which seems that the rate limit is triggered by our frequent cache-updating requests.

To solve this issue, this PR changes the setup-rust steps to use the warmup strategy. This strategy will relax the granularity of the cache keys, and only save caches when the workflow is running on the main branch. (The cache will still be restored in other branches)

I also updated the cache-target options for some jobs, because they're building in release mode but caching the debug target.

Test Plan

CI should pass and the new cache strategy should work as expected.

Copy link

netlify bot commented Mar 22, 2024

Deploy Preview for biomejs ready!

Name Link
🔨 Latest commit dee5bc1
🔍 Latest deploy log https://app.netlify.com/sites/biomejs/deploys/65fdc47bdb979800074a7387
😎 Deploy Preview https://deploy-preview-2166--biomejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 93 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@Sec-ant Sec-ant merged commit faf6587 into biomejs:main Mar 22, 2024
4 of 5 checks passed
@Sec-ant Sec-ant deleted the ci/cache-base branch March 22, 2024 17:55
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.

None yet

1 participant