Skip to content

Make the paybutton server run on ARM64#1134

Open
Fabcien wants to merge 1 commit into
PayButton:masterfrom
Fabcien:arm64
Open

Make the paybutton server run on ARM64#1134
Fabcien wants to merge 1 commit into
PayButton:masterfrom
Fabcien:arm64

Conversation

@Fabcien

@Fabcien Fabcien commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

There is no point hardcoding the platform, it only prevents the app from working on ARM64 which is not desirable.

Summary by CodeRabbit

  • Chores
    • Updated the container base image to use the default Node LTS Alpine image instead of a platform-pinned variant.
    • Simplified the build setup with no other configuration changes.

There is no point hardcoding the platform, it only prevents the app from working on ARM64 which is not desirable.
@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 09226d58-430e-4570-9e2e-9bee165310ba

📥 Commits

Reviewing files that changed from the base of the PR and between 525622e and a7d8fb1.

📒 Files selected for processing (1)
  • Dockerfile

📝 Walkthrough

Walkthrough

The Docker base image was changed from a platform-pinned Node Alpine image to the default node:lts-alpine image.

Changes

Docker base image update

Layer / File(s) Summary
Base image update
Dockerfile
The FROM line no longer pins linux/amd64 and now uses node:lts-alpine without a platform constraint.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I hopped through the Docker mist,
and found a tidier alpine twist.
No platform pin, just a softer breeze,
keeping builds light with easy ease.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is too sparse and omits required template sections like Related to, Description, and Test plan. Fill in the template sections: add the linked issue, a brief description of the change, and a test plan with validation steps.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: removing the platform pin to support ARM64.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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