Skip to content
This repository was archived by the owner on Feb 15, 2026. It is now read-only.

fix(docker): add py3-setuptools for arm builds on node 20#4341

Merged
sct merged 1 commit intodevelopfrom
fix/update-node-20-fix
Feb 15, 2026
Merged

fix(docker): add py3-setuptools for arm builds on node 20#4341
sct merged 1 commit intodevelopfrom
fix/update-node-20-fix

Conversation

@sct
Copy link
Owner

@sct sct commented Feb 15, 2026

Description

No idea if this fixes it but trying anyways

Screenshot (if UI-related)

To-Dos

  • Successful build yarn build
  • Translation keys yarn i18n:extract
  • Database migration (if required)

Issues Fixed or Closed

  • Fixes #XXXX

Summary by CodeRabbit

  • Chores
    • Updated Docker image dependencies for ARM-based builds to include Python setuptools support alongside existing development tools.

Copilot AI review requested due to automatic review settings February 15, 2026 00:47
@coderabbitai
Copy link

coderabbitai bot commented Feb 15, 2026

📝 Walkthrough

Walkthrough

The Dockerfile was modified to include py3-setuptools in the APK package installation for ARM-based target platforms (linux/arm64 and linux/arm/v7), augmenting the existing Python and build toolchain dependencies without altering control flow.

Changes

Cohort / File(s) Summary
Docker ARM Package Configuration
Dockerfile
Added py3-setuptools to the APK package set for ARM targets (linux/arm64, linux/arm/v7) in the RUN instruction, extending the Python build toolchain capabilities.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A setuptools seed, so small but true,
Plants Python power for ARM too,
The Docker image, now more complete,
Makes builds and wheels quite fleet!
hops

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: adding py3-setuptools for ARM builds on Node 20, which directly corresponds to the Dockerfile modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into develop

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/update-node-20-fix

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

Copy link

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 adds the py3-setuptools package to the Docker build dependencies for ARM architectures (arm64 and arm/v7) when running on Node 20. This change aims to address potential build failures related to Python setuptools missing during the native module compilation process with node-gyp.

Changes:

  • Added py3-setuptools to the ARM platform-specific Alpine package installation

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

@sct sct merged commit 98ea135 into develop Feb 15, 2026
13 checks passed
@sct sct deleted the fix/update-node-20-fix branch February 15, 2026 00:55
@github-actions
Copy link

🎉 This PR is included in version 1.35.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants