Skip to content

verify-action-build: show Docker build step summary and add --no-cache flag#653

Merged
potiuk merged 2 commits intomainfrom
verify-build-docker-progress
Apr 4, 2026
Merged

verify-action-build: show Docker build step summary and add --no-cache flag#653
potiuk merged 2 commits intomainfrom
verify-build-docker-progress

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Apr 2, 2026

Summary

  • After Docker build completes, display a concise summary of each build step (with timing and cached status) instead of hiding all output behind a spinner
  • Add --no-cache CLI flag to force a clean Docker build without layer cache

Test plan

  • Run uv run utils/verify-action-build.py <action> and verify step summary appears after build
  • Run with --no-cache and verify Docker rebuilds all layers from scratch
  • Verify a failing Docker build prints full output for diagnosis

Generated with Claude Code

@potiuk potiuk requested a review from raboof April 2, 2026 10:36
@dave2wave dave2wave removed their assignment Apr 2, 2026
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 4, 2026

This one is god for diagnostics and seeing what's going on

@potiuk potiuk requested a review from dave2wave April 4, 2026 09:42
Copy link
Copy Markdown
Member

@raboof raboof left a comment

Choose a reason for hiding this comment

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

No strong objections, so OK to merge AFAICS, but some comments for your consideration.

- Rename no_cache param to cache (positive boolean) per reviewer preference
- Show Docker build step summary only on failure by default
- Add --show-build-steps flag to display steps on success too
- Document --no-cache and --show-build-steps flags in README

Generated-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@potiuk potiuk merged commit 928b06c into main Apr 4, 2026
7 checks passed
@potiuk potiuk deleted the verify-build-docker-progress branch April 4, 2026 14:03
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 4, 2026

implemented both comments.

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.

3 participants