Skip to content

build: Update buildkit to v0.25.2 - #2242

Merged
kroese merged 3 commits into
masterfrom
build
Oct 5, 2026
Merged

kroese merged 3 commits into
masterfrom
build

Conversation

@kroese

@kroese kroese commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI balanced review requested due to automatic review settings October 5, 2026 17:53

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2034:warning:173:3: attempt appears unused. Verify use (or export if used externally) [shellcheck]

for attempt in {1..30}; do

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2028:info:395:12: echo may not expand escape sequences. Use printf [shellcheck]

echo "Failed to read C:\\OEM\\validation.token."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2028:info:404:12: echo may not expand escape sequences. Use printf [shellcheck]

echo "Failed to read \\\\host.lan\\Data\\validation.token."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2028:info:504:12: echo may not expand escape sequences. Use printf [shellcheck]

echo "Failed to read C:\\OEM\\validation.token."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2028:info:504:12: echo may not expand escape sequences. Use printf [shellcheck]

echo "Failed to read C:\\OEM\\validation.token."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2028:info:513:12: echo may not expand escape sequences. Use printf [shellcheck]

echo "Failed to read \\\\host.lan\\Data\\validation.token."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2028:info:513:12: echo may not expand escape sequences. Use printf [shellcheck]

echo "Failed to read \\\\host.lan\\Data\\validation.token."

@kroese
kroese merged commit 8fa0dac into master Oct 5, 2026
3 checks passed
@kroese
kroese deleted the build branch October 5, 2026 17:55

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.

Copilot review overview

🟢 Approval recommended

The Buildx configuration and build invocation are valid and consistently use the pinned BuildKit version.

Review effort: Balanced
Findings: None

What changed in this PR

Updates validation image builds to use BuildKit v0.25.2.

Changes:

  • Configures Docker Buildx with the pinned BuildKit image.
  • Builds and loads the image using the configured builder with visible logs.
File Description
.github/​workflows/​install.yml Pins BuildKit and migrates validation builds to Buildx.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

2 participants