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: Update npm cache configuration for faster builds #351

Merged
merged 2 commits into from
May 24, 2024

Conversation

detautama
Copy link
Contributor

To improve build performance, Next.js saves a cache to .next/cache that is shared between builds.

Reference: https://nextjs.org/docs/pages/building-your-application/deploying/ci-build-caching

@detautama detautama added the priority:low 🏖️ Low priority issue label May 22, 2024
@detautama detautama self-assigned this May 22, 2024
@DennisAlund
Copy link
Member

Thank you. It's already implemented in the next branch

- name: Cache hosting build
uses: actions/cache@v4
id: cache-hosting
with:
key: ${CI_COMMIT_REF_SLUG}
path: |
./hosting/node_modules/
./hosting/.next/cache/

@DennisAlund
Copy link
Member

Apologies, I stand corrected seeing that you implemented your solution according to the docs 🙏

https://nextjs.org/docs/pages/building-your-application/deploying/ci-build-caching#github-actions

@DennisAlund DennisAlund reopened this May 24, 2024
Copy link

Visit the preview URL for this PR (updated for commit ae813ea):

https://tanam-testing--pr351-continuous-integrati-uihesui5.web.app

(expires Fri, 31 May 2024 02:21:39 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 5bbe8a1a68d9684d656bffba10008fe532204561

@DennisAlund DennisAlund merged commit 1a66f0f into next May 24, 2024
3 checks passed
@DennisAlund DennisAlund deleted the Continuous-Integration-(CI)-Build-Caching branch May 24, 2024 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low 🏖️ Low priority issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants