-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Zaber motion #10332
Zaber motion #10332
Conversation
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/zaber-motion:
|
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 just wanted to let you know that I linted all conda-recipes in your PR ( |
Seems like we need macos 10.10 for go. |
ba58484
to
ea78069
Compare
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/staged-recipes this is ready for review. |
Looks like it is failing on OSX still 🙍♂️ |
it will fail until it gets merged. It needs a newer OSX than the one on the staged-recipes. |
@@ -0,0 +1,4 @@ | |||
macos_min_version: # [osx] | |||
- 10.10 # [osx] |
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.
@scopatz this should rerender things correctly
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! I am happy for you to figure this out in a feedstock
In talking to upstream, i got them to fixup their code to point to a public repo for their go module.
Needs: #10515
OSX will work after re-rendering since it needs 10.10 instead of 10.9: https://github.com/ramonaoptics/zaber-motion-feedstock
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details)