-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FIX: Set python_min of Python 3.8 #59
FIX: Set python_min of Python 3.8 #59
Conversation
* Use 'python {{ python_min }}' syntax for the python requirements for noarch python recipes. - c.f. https://conda-forge.org/docs/maintainer/knowledge_base/#noarch-python * Add recipe/conda_build_config.yaml to override the global python_min with hatchling's python_min of 3.8. * Install with recommended --no-deps --no-build-isolation options to ensure build dependencies are correct. * Update to 'pypi.org'. * Bump build number.
@conda-forge-admin, please rerender |
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 ( For recipe/meta.yaml:
|
…nda-forge-pinning 2024.11.07.16.38.57
I assume that build backends are immune from this. I'm not sure if there is some way to indicate to conda-forge that they are self sufficient. |
@conda-forge/hatchling this is ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thank you!
Thanks @ofek. Can I also get you to 👍 conda-forge/conda-forge-repodata-patches-feedstock#901 ? |
Description
python {{ python_min }}
syntax for the python requirements for noarch python recipes.recipe/conda_build_config.yaml
to override the globalpython_min
withhatchling
'spython_min
of3.8
.--no-deps --no-build-isolation
options to ensure build dependencies are correct.pypi.org
.Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)