Skip to content

Require setuptools<82 for xarray-schema#1151

Merged
h-vetinari merged 1 commit into
conda-forge:mainfrom
jdblischak:xarray-schema-setuptools
Feb 12, 2026
Merged

Require setuptools<82 for xarray-schema#1151
h-vetinari merged 1 commit into
conda-forge:mainfrom
jdblischak:xarray-schema-setuptools

Conversation

@jdblischak
Copy link
Copy Markdown
Member

@jdblischak jdblischak commented Feb 10, 2026

Checklist

  • Used a static YAML file for the patch if possible (instructions).
  • Ran pre-commit run -a and ensured all files pass the linting checks.
  • Ran python show_diff.py and posted the output as part of the PR.
  • Modifications won't affect packages built in the future.

Dealing with the problem of pkg_resources being removed in setuptools 82

@jdblischak
Copy link
Copy Markdown
Member Author

================================================================================
================================================================================
noarch
noarch::xarray-schema-0.0.3-pyhd8ed1ab_0.tar.bz2
-    "numpy >=1.20",
+    "numpy >=1.21",
+    "setuptools <82",

@conda-forge-admin
Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@jdblischak
Copy link
Copy Markdown
Member Author

Note: had to use add_depends because setuptools was only in host requirements, but pkg_resources is imported in __init__.py, and thus this needs to be enforced at runtime too

@h-vetinari
Copy link
Copy Markdown
Member

conda-forge/xarray-schema-feedstock#2 should be merged first, if that doesn't happen within 1-2 days due to lack of response, I'm willing to pull this forward.

@h-vetinari h-vetinari merged commit 8329043 into conda-forge:main Feb 12, 2026
3 checks passed
@jdblischak jdblischak deleted the xarray-schema-setuptools branch February 12, 2026 18:06
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