Skip to content

Ensure base image is refreshed for builds#18

Open
tekante wants to merge 1 commit intomasterfrom
include-base-image-updates
Open

Ensure base image is refreshed for builds#18
tekante wants to merge 1 commit intomasterfrom
include-base-image-updates

Conversation

@tekante
Copy link
Member

@tekante tekante commented Feb 5, 2026 •
edited by coderabbitai bot
Loading

While working on the automatic updates action I noticed that the composer in our images seemed to be out of date. After some digging I think our builds must be using a cached base image (or something else cached in the build). These two flags seem to take care of it.

I couldn't find documentation that seemed definitive that just setting pull to true but leaving no-cache false would update things if the base image changed so I set them both to true as we only build these once a week.

Summary by CodeRabbit

  • Chores
    • Updated build process to always pull the latest base images and bypass caching for consistent, up-to-date builds. This may increase build duration but ensures the latest dependencies are used.

@tekante tekante requested a review from mike-potter February 5, 2026 14:57
@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026 •
edited
Loading

Walkthrough

The pull request modifies the GitHub Actions workflow configuration to add two options to the Docker Buildx build step: pull: true to always fetch the latest base images and no-cache: true to bypass build cache, ensuring fresh builds without cached layers.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/default.yml		 Added pull: true and no-cache: true options to Docker Buildx build step to force fresh base image pulls and disable build caching.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A cache is cleared, base images shine,
Fresh pulls each time, the builds align,
No stale layers hiding in the dark,
Every build leaves a pristine mark! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately reflects the main change: adding pull: true and no-cache: true flags to ensure the base image is always refreshed for builds.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch include-base-image-updates

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mike-potter mike-potter Awaiting requested review from mike-potter

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tekante