-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Adding osqp-eigen recipe #15309
Adding osqp-eigen recipe #15309
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/osqp-eigen:
For recipes/osqp-eigen:
Documentation on acceptable licenses can be found here. |
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/stable-baselines3:
|
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 ( |
58c2d1c
to
1cee4a0
Compare
@GiulioRomualdi feel free to add me as well as maintainer, it is a good practice to avoid having just one maintainer in recipes to reduce the bus factor. |
The CI is now green, @conda-forge/help-c-cpp this is now ready for review. |
Co-authored-by: Silvio Traversaro <[email protected]>
@wolfv just FYI, this is the first package that we migrate from the |
Awesome! |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).The PR is ready to be reviewed @conda-forge/help-c-cpp
This PR closes #15080
cc @traversaro