-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
addingpymedm recipe
#32155
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
addingpymedm recipe
#32155
Conversation
|
Hi! This is the staged-recipes linter and your PR looks excellent! 🚀 |
|
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 ( |
|
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/pymedm/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/21842207259. Examine the logs at this URL for more detail. |
|
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 ( |
|
@conda-forge/help-python This Windows build giving me trouble, and I think I am missing something obvious for adding in some logic that simply skips Windows. In theory, we support Windows, but in practice we do not (and never have) due to the |
The problem is that we do not have jaxlib on Windows. Leave it noarch b/c then that dependency is available, this package will be ready without the need to do anything here. |
|
Yeah, I was digging around the the logs and thought that was the case. So is the solution here to simply allow the Windows build to fail? And if so, should I add any extra logic in to make the check green? |
Yep. It won't be installable on Windows until that dependency can be satisfied. |
|
@ocefpaf, thanks! You are the best! |
Checklist
url) rather than a repo (e.g.git_url) is used in your recipe (see here for more details).