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

[Docs Site] Cache Astro assets in ci.yml #17487

Merged
merged 2 commits into from
Oct 15, 2024
Merged

[Docs Site] Cache Astro assets in ci.yml #17487

merged 2 commits into from
Oct 15, 2024

Conversation

KianNH
Copy link
Contributor

@KianNH KianNH commented Oct 11, 2024

Summary

Cache Astro assets (resized images) like we do in publish-production and publish-preview.

Explicit NPM cache step is unnecessary, v4 will handle this.

Copy link

cloudflare-workers-and-pages bot commented Oct 11, 2024

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 83beabb
Status: ✅  Deploy successful!
Preview URL: https://8f0b92eb.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://kian-pcx-14089.cloudflare-docs-7ou.pages.dev

View logs

@GregBrimble
Copy link
Contributor

GregBrimble commented Oct 11, 2024

Pages just caches the entire .astro directory. Any reason not to do that here? https://developers.cloudflare.com/pages/configuration/build-caching/#frameworks

Explicit NPM cache step is unnecessary, v4 will handle this.

TIL. Good addition, GitHub!

@KianNH
Copy link
Contributor Author

KianNH commented Oct 15, 2024

Pages just caches the entire .astro directory. Any reason not to do that here? https://developers.cloudflare.com/pages/configuration/build-caching/#frameworks

No particular reason! Currently Astro only caches assets between runs but with Astro 5 on the horizon there'll be more things in there related to incremental builds so I'll move it over to the whole folder.

@KianNH KianNH merged commit d717b6a into production Oct 15, 2024
11 checks passed
@KianNH KianNH deleted the kian/PCX-14089 branch October 15, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants