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: enables github docker cache #8187

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Conversation

chriswk
Copy link
Contributor

@chriswk chriswk commented Sep 19, 2024

So, since we updated to build-push-action@v6 we're now getting a nice summary of the build on the summary page of the action. It pointed out that we're caching 0% of our docker build. Since we probably change our own code more often than we change our dependencies, sharing the initial yarn install step between builds from cache seems to be a good idea.

@chriswk chriswk self-assigned this Sep 19, 2024
Copy link

vercel bot commented Sep 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
unleash-docs ⬜️ Ignored (Inspect) Sep 19, 2024 0:02am
unleash-monorepo-frontend ⬜️ Ignored (Inspect) Sep 19, 2024 0:02am

Copy link
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

@chriswk chriswk enabled auto-merge (squash) September 19, 2024 12:38
Copy link
Contributor

@gastonfournier gastonfournier left a comment

Choose a reason for hiding this comment

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

🏅

@chriswk chriswk merged commit 6d51213 into main Sep 19, 2024
9 checks passed
@chriswk chriswk deleted the chore/addCachingToDockerBuilds branch September 19, 2024 12:55
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.

2 participants