Skip to content

build(medcat-v1): CU-869c41zvh Fix setuptools for MedCAT v1#331

Merged
mart-r merged 2 commits intomainfrom
build/medcat-v1/CU-869c41zvh-fix-setuptools
Feb 12, 2026
Merged

build(medcat-v1): CU-869c41zvh Fix setuptools for MedCAT v1#331
mart-r merged 2 commits intomainfrom
build/medcat-v1/CU-869c41zvh-fix-setuptools

Conversation

@mart-r
Copy link
Collaborator

@mart-r mart-r commented Feb 12, 2026

This PR fixes the issue where setuptools>=82.0.0 have completely removed pkg_resources. So in order to avoid running into that issue, the PR limits setuptools to less than 82.

This wouldn't normally be a good option (i.e this avoids getting future bug or security fixes), but v1 is legacy so I'd say it's fine.

PS:
The PR is prompted by the failing workflow, e.g:
https://github.com/CogStack/cogstack-nlp/actions/runs/21940579910

This PR fixes the issue where `setuptools>=82.0.0` have completely removed `pkg_resources`. So in order to avoid running into that issue, the PR limits `setuptools` to less than 82.

This wouldn't normally be a good option (i.e this avoids getting future bug or security fixes), but v1 is legacy so I'd say it's fine.
@tomolopolis
Copy link
Member

@mart-r mart-r merged commit f84b916 into main Feb 12, 2026
17 checks passed
@mart-r mart-r deleted the build/medcat-v1/CU-869c41zvh-fix-setuptools branch February 12, 2026 16:10
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.

3 participants