fix(amplify): cap setuptools<82 to preserve pkg_resources#1574
Conversation
Signed-off-by: svc-bionemo <267129667+svc-bionemo@users.noreply.github.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
Description
Fixes nightly CI failure in
unit-tests (models/amplify)(run #25911784459).Root Cause
When
pip install -e .runs for the amplify recipe, pip resolvessetuptools>=61.0to v82.0.1. In setuptools 82+,pkg_resourceswas removed as a bundled module. Thehypothesistesting library (installed in the base container image) importspkg_resourcesin itsentry_points.py, causing pytest to crash at startup:Fix
setuptools>=61.0,<82in[build-system].requiressetuptools<82to[project].dependenciesto prevent runtime upgradeThis preserves backward compatibility until either the base image is updated with a newer hypothesis that does not use
pkg_resources, or setuptools-pkg-resources is added.Type of changes
Signed-off-by: svc-bionemo 267129667+svc-bionemo@users.noreply.github.com