Skip to content

github-builder: add shared exporter construction with compression - #1397

Merged
crazy-max merged 1 commit into
docker:mainfrom
crazy-max:github-builder-output-compression
Oct 27, 2026
Merged

crazy-max merged 1 commit into
docker:mainfrom
crazy-max:github-builder-output-compression

Conversation

@crazy-max

Copy link
Copy Markdown
Member

relates to docker/github-builder#207

Centralize image and local exporter settings for the build and bake workflows. Preserve existing defaults and support optional compression settings with CSV quoting for exporter values.

Centralize image and local exporter settings for the build and bake workflows.
Preserve existing defaults and support optional compression settings with
CSV quoting for exporter values.

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
crazy-max force-pushed the github-builder-output-compression branch from 07ce423 to bc965e8 Compare October 27, 2026 09:03
@crazy-max
crazy-max requested a review from a team October 27, 2026 09:03
@vvoland
vvoland requested a balanced review from Copilot October 27, 2026 14:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused helper, dependency updates, and supplied test coverage have no identified blocking issues.

No open issues 🎉

What changed in this PR

Adds a shared exporter-construction helper for GitHub Builder’s build and Bake workflows, preserving existing exporter defaults while supporting optional image compression and CSV-safe values.

Changes:

  • Adds image and local exporter construction with input validation and optional compression settings.
  • Tests defaults, compression options, CSV escaping, and invalid inputs.
  • Adds and locks the csv-stringify dependency.
File Description
yarn.lock Locks the new CSV serialization dependency.
src/​github-builder/​build-output.ts Implements shared exporter construction and compression options.
package.json Adds csv-stringify as a runtime dependency.
__tests__/​github-builder/​build-output.test.ts Covers exporter policies, compression, escaping, and validation.

Review effort: Balanced

@crazy-max
crazy-max merged commit 8d0241e into docker:main Oct 27, 2026
117 checks passed
@crazy-max
crazy-max deleted the github-builder-output-compression branch October 27, 2026 15:09
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