Skip to content

feat: add --no-cache flag to container:push (W-23597904, #1390) - #3840

Merged
sbosio merged 2 commits into
mainfrom
W-23597904
Aug 12, 2026
Merged

feat: add --no-cache flag to container:push (W-23597904, #1390)#3840
sbosio merged 2 commits into
mainfrom
W-23597904

Conversation

@heroku-johnny

Copy link
Copy Markdown
Contributor

Summary

  • Adds --no-cache boolean flag to heroku container:push
  • Passes --no-cache to the underlying docker build command when the flag is set
  • Adds noCache?: boolean to the BuildImageParams type in DockerHelper

Test plan

  • npm run test:file test/unit/lib/container/docker-helper.unit.test.ts — 2 new tests for noCache: true and noCache: false
  • npm run test:file test/unit/commands/container/push.unit.test.ts — 1 new test verifying --no-cache is passed through to buildImage
  • npx eslint src/lib/container/docker-helper.ts src/commands/container/push.ts — no errors

Closes #1390

🤖 Generated with Claude Code

@michaelmalave michaelmalave left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Added a comment on user facing text added. Otherwise, code passes. LGTM

Comment thread src/commands/container/push.ts Outdated
@sbosio
sbosio requested a review from heicheng18 August 11, 2026 20:49
@sbosio
sbosio deployed to AcceptanceTests August 11, 2026 20:52 — with GitHub Actions Active
@sbosio
sbosio deployed to AcceptanceTests August 11, 2026 20:52 — with GitHub Actions Active
@sbosio
sbosio deployed to AcceptanceTests August 11, 2026 20:52 — with GitHub Actions Active
@sbosio
sbosio deployed to AcceptanceTests August 11, 2026 20:52 — with GitHub Actions Active
heroku-johnny and others added 2 commits August 12, 2026 17:12
Adds a --no-cache boolean flag to `heroku container:push` that passes
`--no-cache` to the underlying `docker build` command, allowing users to
bypass the Docker layer cache on demand. Resolves GitHub issue #1390.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@sbosio
sbosio deployed to AcceptanceTests August 12, 2026 20:17 — with GitHub Actions Active
@sbosio
sbosio deployed to AcceptanceTests August 12, 2026 20:17 — with GitHub Actions Active
@sbosio
sbosio deployed to AcceptanceTests August 12, 2026 20:17 — with GitHub Actions Active
@sbosio
sbosio deployed to AcceptanceTests August 12, 2026 20:17 — with GitHub Actions Active
@sbosio
sbosio merged commit 91a0c42 into main Aug 12, 2026
19 checks passed
@sbosio
sbosio deleted the W-23597904 branch August 12, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add way to set docker build to use --no-cache

4 participants