-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[WIP] add minigun #12540
[WIP] add minigun #12540
Conversation
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
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 ( |
@@ -0,0 +1,46 @@ | |||
{% set name = "minigun" %} | |||
{% set version = "0.0.1.dev0" %} |
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.
Is there a fully released version to build from?
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.
Ideally, we should wait to have a git tag before merging this. See dglai/minigun#14
@scopatz thanks for the review but this is WIP (sorry I should have mentioned it). Will ping again when I am done here. |
Not sure this lib really make sense since it is only used in dgl and is headers only. Closing here. |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details)