Skip to content

Conversation

@planv
Copy link
Contributor

@planv planv commented Dec 11, 2025

fixing build issue
image

Copilot AI review requested due to automatic review settings December 11, 2025 20:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes multi-architecture build errors in the Dockerfile by improving architecture detection and checksum verification for Node.js installation.

Key changes:

  • Uses Docker's TARGETARCH build argument for more reliable architecture detection across platforms
  • Adds debug output to verify detected architecture during builds
  • Refactors checksum verification to use an intermediate file for better error handling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@planv planv requested a review from a team December 11, 2025 20:46
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 11, 2025 20:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@planv planv merged commit d316527 into latest Dec 12, 2025
7 checks passed
@planv planv deleted the dependency-upgrade branch December 12, 2025 12: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.

4 participants