Skip to content

build/bake: replace inline logic with shared runtime helpers - #297

Open
crazy-max wants to merge 1 commit into
docker:mainfrom
crazy-max:use-runtime-helpers
Open

crazy-max wants to merge 1 commit into
docker:mainfrom
crazy-max:use-runtime-helpers

Conversation

@crazy-max

Copy link
Copy Markdown
Member

This adopts the helpers introduced in docker/actions-toolkit#1333 and the index annotation conversion from docker/actions-toolkit#1332, released in v0.101.0. Runner mapping, registry identity parsing, Bake target validation, and annotation conversion now use the shared runtime implementations instead of inline workflow code, preserving behavior for supported inputs.

Workflow orchestration and outputs remain unchanged. This removes duplicated logic and the private js-yaml dependency lookup. The only intentional diagnostic changes are reduced informational logging and registry YAML errors reporting their location without exposing input excerpts.

Use the runtime helpers for runner mapping, registry identity parsing, Bake target validation and index
annotation conversion. Keep workflow orchestration and outputs unchanged while removing duplicated parsing
code and the private js-yaml dependency lookup.

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
crazy-max marked this pull request as ready for review September 18, 2026 16:46
@crazy-max
crazy-max requested a review from a team as a code owner September 18, 2026 16:46
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.

1 participant