Skip to content

Add arm64 support to CI Docker builds#2

Open
ChrisLovering wants to merge 6 commits into
mainfrom
ci/arm64-builds
Open

Add arm64 support to CI Docker builds#2
ChrisLovering wants to merge 6 commits into
mainfrom
ci/arm64-builds

Conversation

@ChrisLovering

Copy link
Copy Markdown
Member

Builds and pushes multi-platform Docker images for both amd64 and arm64.

@ChrisLovering ChrisLovering requested a review from jb3 June 23, 2026 21:33
Comment thread versions.toml

[config]
versions = [
"3.14",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why was this removed?

Comment on lines +54 to +55
- name: Set up QEMU
uses: docker/setup-qemu-action@v4

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is this still needed?

ghcr.io/python-discord/python-builds:builder-base
ghcr.io/python-discord/python-builds:builder-base-${{ needs.pyenv-version.outputs.pyenv_version }}

- name: Run Docker container to resolve versions

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Was there no reason to run this script inside Docker in the first place?

Comment on lines +100 to +101
ghcr.io/python-discord/python-builds:${{ inputs.tag }}-linux-amd64 \
ghcr.io/python-discord/python-builds:${{ inputs.tag }}-linux-arm64

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is there a way to avoid hard-coding the platforms here?

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