-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
add ismrmrd-python package #25854
add ismrmrd-python package #25854
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 was successfully able to build & test this on a windows box. Retrying CI |
@conda-forge-admin, please ping team Hi, are there any insights into MemoryErrors on conda-forge CI infra? Can we still proceed? |
Hi! This is the friendly automated conda-forge-webservice. I was asked to ping @conda-forge/staged-recipes and so here I am doing that. |
To help direct your pull request to the best reviewers, please mention a topic-specifc team if your recipe matches any of the following: conda-forge/help-c-cpp, conda-forge/help-cdts, conda-forge/help-go, conda-forge/help-java, conda-forge/help-julia, conda-forge/help-nodejs, conda-forge/help-perl, conda-forge/help-python, conda-forge/help-python-c, conda-forge/help-r, conda-forge/help-ruby,or conda-forge/help-rust. Thanks! |
B/c this is noarch yes, we can proceed and the package will be built and test on Linux but available to all archs. However, if you want to make sure most of the tests passes on Windows anyway, you can skip that one. This is not mandatory to merge though and I would recommend you to tinker with this later. |
Co-authored-by: Filipe <[email protected]>
Thanks a lot for the review & merge! I will look into ironing out those in the future. |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).xref ismrmrd/ismrmrd-python#69