-
-
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
add xvega #12522
add xvega #12522
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/xvega:
For recipes/xvega:
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 ( |
Thanks for the thorough review @scopatz. |
I was a bit verbose/explicit with dependencies. I base them on what is imported by modules within the package. `dgl` could be added as a dependency if it's added to conda-forge. See dmlc/dgl#1855 and #12522 Even though this is a pure Python package, I run a few of the fastest tests that exercise different files, because this package has some heavy dependencies (pytorch, pytorch_geometric, etc) and I want that warm fuzzy feeling :) I get the tarball from github instead of PyPI, because the PyPI tarball does not (yet) contain the license file or the tests. Currently, the github tarball is not that much larger than from PyPI.
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details)