-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Use noarch: python
for Python packages
#128
Use noarch: python
for Python packages
#128
Conversation
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 ( I do have some suggestions for making it better though... For recipe:
|
…nda-forge-pinning 2023.08.18.06.22.42
As the XGBoost Python packages are pure Python (using `ctypes` to interact with the `libxgboost` library), there is no need to build these for every Python version and architecture. So simply mark these as `noarch: python` to get the minimum needed builds and move on.
@conda-forge-admin, please re-render |
…nda-forge-pinning 2023.08.18.06.22.42
@hcho3 , could you please review? 🙂 |
We could consider dropping some of the Python mutex packages at this point since really only selecting |
@conda-forge-admin, please re-render |
…nda-forge-pinning 2023.08.25.17.04.40
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
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 ( I do have some suggestions for making it better though... For recipe:
|
As the XGBoost Python packages are pure Python (using
ctypes
to interact with thelibxgboost
library), there is no need to build these for every Python version and architecture. So simply mark these asnoarch: python
to get the minimum needed builds and move on.Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #127.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.
Fixes #127