Skip to content

Update Docker deploy CI to modern actions - #14

Merged
mmore500 merged 4 commits into
masterfrom
claude/update-docker-ci-dthTf
Feb 13, 2026
Merged

mmore500 merged 4 commits into
masterfrom
claude/update-docker-ci-dthTf

Conversation

@mmore500

Copy link
Copy Markdown
Owner
  • Upgrade actions/checkout from v2 to v4
  • Add explicit GHCR login step via docker/login-action@v3
  • Add Docker Buildx setup via docker/setup-buildx-action@v3
  • Add dynamic tag generation with full ghcr.io image paths
  • Upgrade docker/build-push-action from v1 to v6
  • Use env.TAGS and context-based push condition

https://claude.ai/code/session_01YWwHVNCxDxmmiCT9Gmjuop

- Upgrade actions/checkout from v2 to v4
- Add explicit GHCR login step via docker/login-action@v3
- Add Docker Buildx setup via docker/setup-buildx-action@v3
- Add dynamic tag generation with full ghcr.io image paths
- Upgrade docker/build-push-action from v1 to v6
- Use env.TAGS and context-based push condition

https://claude.ai/code/session_01YWwHVNCxDxmmiCT9Gmjuop
- Upgrade actions/checkout from v2 to v4
- Add docker/setup-buildx-action@v3 step
- Upgrade docker/build-push-action from v1 to v6
- Add context and remove deprecated add_git_labels

Fixes "client version 1.40 is too old" error caused by
docker/build-push-action@v1 being incompatible with newer Docker.

https://claude.ai/code/session_01YWwHVNCxDxmmiCT9Gmjuop
The python:3.10-slim base image ships with an old pip (23.0.1) that
doesn't include setuptools. The setuptools_scm build dependency
imports pkg_resources from setuptools, causing a ModuleNotFoundError.

https://claude.ai/code/session_01YWwHVNCxDxmmiCT9Gmjuop
setuptools_scm >= 2.0.0, <3 imports from pkg_resources, which is
incompatible with setuptools >= 82. Since the version is statically
set in pyproject.toml (not dynamically via SCM), setuptools_scm is
unnecessary. cython is also unused (no Cython code in the project).

https://claude.ai/code/session_01YWwHVNCxDxmmiCT9Gmjuop

@mmore500 mmore500 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

lgtm

@mmore500
mmore500 merged commit 9c40ccd into master Feb 13, 2026
10 checks passed
@mmore500
mmore500 deleted the claude/update-docker-ci-dthTf branch February 13, 2026 00:03
mmore500 pushed a commit to mmore500/downstream that referenced this pull request Feb 13, 2026
- Add pip/setuptools upgrade step in Dockerfile before installing packages
- Remove unused cython and setuptools_scm build dependencies from pyproject.toml

Replicates mmore500/joinem#14.

https://claude.ai/code/session_01Do3n1osihQbx3yJGF5Ncug
mmore500 added a commit to mmore500/downstream that referenced this pull request Feb 13, 2026
* Update Docker deploy CI to use modern action versions

- Upgrade actions/checkout from v2 to v4
- Replace docker/build-push-action v1 with v6
- Add explicit docker/login-action v3 for GHCR authentication
- Add docker/setup-buildx-action v3
- Use fully qualified image tags (ghcr.io/mmore500/downstream)
- Remove deprecated repository/registry/add_git_labels options

https://claude.ai/code/session_01Do3n1osihQbx3yJGF5Ncug

* Fix Docker build by upgrading pip and removing unused build deps

- Add pip/setuptools upgrade step in Dockerfile before installing packages
- Remove unused cython and setuptools_scm build dependencies from pyproject.toml

Replicates mmore500/joinem#14.

https://claude.ai/code/session_01Do3n1osihQbx3yJGF5Ncug

* Fix Dockerfile pip upgrade on Debian-managed Python

Use --force-reinstall instead of --upgrade to avoid "Cannot uninstall
pip, RECORD file not found" error with Debian-packaged pip.

https://claude.ai/code/session_01Do3n1osihQbx3yJGF5Ncug

* Fix Dockerfile: only upgrade setuptools, skip pip

Debian-managed pip lacks a RECORD file and cannot be uninstalled or
force-reinstalled. Since uv handles all package installation, only
setuptools needs upgrading.

https://claude.ai/code/session_01Do3n1osihQbx3yJGF5Ncug

---------

Co-authored-by: Claude <noreply@anthropic.com>
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