-
Notifications
You must be signed in to change notification settings - Fork 7
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
Begin loosening requirements #22
Conversation
@AbdealiJK for your awareness, I'm working on #21 here! Feedback welcome. |
Just a thought - while doing changes to the GitHub workflow, you may also want to add black for code formatting to keep the syntax styles consistent Also, in pypi I saw manylinux wheels. But I wasn't clear on why that is the case. Cause grand seems to be a pure python package ... And so, it should not be dependent on OS right ? EDIT: Ignore my above statement on manylinux. I checked grand instead of grand-graph by mistake. For grand-graph, it may be good to add bdist_wheel and upload wheels directly to pypi as they are being standardized |
Yes! Probably going to do this in a separate PR so that the inevitable code changes don't get mixed in with the import sorting in this PR.
Agreed! |
Fixes #21