-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
MNT: Use noarch python {{ python_min }} variable #35
MNT: Use noarch python {{ python_min }} variable #35
Conversation
@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 ( |
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.
@chrisburr as this is noarch
why are there multiple Python builds happening and not just one?
6e5187f
to
47a011d
Compare
@conda-forge-admin, please rerender |
Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub Actions workflow below for more details. You can also ping The following suggestions might help debug any issues:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11958685116. Examine the logs at this URL for more detail. |
* 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 * Use a Jinja2 set statement for python_min to keep all the build metadata contained in the recipe/meta.yaml and override the global python_min with hist's python_min of 3.8. * Use '{{ PYTHON }}' Jinja2 variable. * Add '--no-build-isolation' pip install flag given multiple outputs. * Bump build number.
…nda-forge-pinning 2024.11.18.16.56.22
recipe/meta.yaml
Outdated
outputs: | ||
- name: hist-base | ||
build: | ||
noarch: python | ||
entry_points: | ||
- hist=hist.classichist:main | ||
script: python -m pip install . --no-deps -vv | ||
# c.f. https://github.com/conda/grayskull/issues/582#issuecomment-2482370940 | ||
script: {{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation |
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.
Hm. I guess {{ PYTHON }}
won't work here, even though this is noarch: python
. Maybe it is related to why there are multiple Python builds?
47a011d
to
e553b82
Compare
@conda-forge-admin, please rerender |
…nda-forge-pinning 2024.11.21.16.17.13
I'm realizing now that this feedstock doesn't behave like |
@conda-forge-admin, please rerender |
…nda-forge-pinning 2024.11.21.20.26.34
Thanks for the useful advice on the top level |
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)