Conversation
…nda-forge-pinning 2025.05.12.15.56.23
regro-cf-autotick-bot
requested review from
isuruf,
maresb and
saraedum
as code owners
|
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 ( |
|
The failing test seems like a temporary CI issue to me. @conda-forge-admin, please rerender |
|
Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like there was nothing to do. This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/15462191372. Examine the logs at this URL for more detail. |
maresb
left a comment
maresb
left a comment
There was a problem hiding this comment.
@tobiasdiez, I reran the failed run and now CI is green. I have not checked for dependency changes.
|
Awesome, thanks! The changes in v2.2.2 are mostly for compatibility with the newest version of Cython, see sagemath/cypari2@2.2.1...2.2.2 |
|
Cool! I'll leave it to you to merge this once you think it's ready. |
|
Thanks for the trust but I'm not a maintainer here ;-) |
It's been missing for a very long time
|
@tobiasdiez, given how involved you are, could I make you a maintainer? |
|
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
Thanks for the offer, but for now I'm happy to be just a 'normal' outside contributor. |
sagemathgh-39944: Include optional packages in `pyproject.toml` and conda environments <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Adds the missing information about optional dependencies in the `pyproject.toml` and use this updated info to include these packages also in the conda environment. ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [ ] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> - (Upstream conda/grayskull#612) - conda-forge/cypari2-feedstock#57 URL: sagemath#39944 Reported by: Tobias Diez Reviewer(s): Isuru Fernando
4 participants
It is very likely that the current package version for this feedstock is out of date.
Checklist before merging this PR:
license_fileis packagedInformation about this PR:
@conda-forge-admin,please add bot automergein the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.bot-rerunlabel to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase@conda-forge-admin, please rerun botin a PR comment to have theconda-forge-adminadd it for you.This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/cf-scripts/actions/runs/14980384271 - please use this URL for debugging.