-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
lightgbm v4.6.0 #70
lightgbm v4.6.0 #70
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 ( I do have some suggestions for making it better though... For recipe/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/13342764913. Examine the logs at this URL for more detail. |
…nda-forge-pinning 2025.02.15.02.01.29
@@ -80,7 +80,7 @@ outputs: | |||
- dask >=2.0.0 | |||
- pandas >=0.24.0 | |||
- pyarrow >=6.0.1 | |||
- scikit-learn !=0.22.0 | |||
- scikit-learn >=0.24.2 |
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.
Ok after a bit of fidgeting with the patches, I think this is working. Will leave it up for a few days until another maintainer has a chance to review. |
This has been up for a couple days and I think the changes are non-controversial, so going to put this in. Please |
Updates to
lightgbm
v4.6.0, which I just released to PyPI:Also proposes moving patches into a
patches/
directory and numbering them, similar to watch is done inarrow-cpp-feedstock
: https://github.com/conda-forge/arrow-cpp-feedstock/tree/main/recipe/patches. I think that is helpful as the number of patches grows in the future.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)