Skip to content

feat: caching containers - #61

Merged
kuraisle merged 2 commits into
mainfrom
feat-cache
Sep 23, 2026
Merged

kuraisle merged 2 commits into
mainfrom
feat-cache

Conversation

@kuraisle

Copy link
Copy Markdown
Member
✨ Feature

PR Description

I've added caching behaviour to the container build workflow.
The way caching works with docker/build-push-action is that the action has cache-from and cache-to fields under with.

The way I've chosen to add this to the workflow is to add string inputs for cache-from and cache-to. There are lots of options to choose; I have used what works with the omop-r-tools build, which is also in docker's example.

I had worried that passing arguments as a string, or passing an empty string, would cause strange behaviour. I tested this in the omop-r-tools repo:

Related Issues or other material

Related #43
Closes #43

✅ Added/updated tests?

  • This PR contains relevant tests / Or doesn't need to per the below explanation

@kuraisle kuraisle changed the title Update publish-container.yml feat: caching containers Sep 23, 2026
@kuraisle
kuraisle marked this pull request as ready for review September 23, 2026 14:19
@kuraisle
kuraisle merged commit e7fef64 into main Sep 23, 2026
10 of 11 checks passed
@kuraisle
kuraisle deleted the feat-cache branch September 23, 2026 14:31
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

Docker build cache

2 participants