-
Notifications
You must be signed in to change notification settings - Fork 440
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
Remove poetry #24
Remove poetry #24
Conversation
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.
Can you either link a PR to what wiki changes would need to be made after this merges, or include those changes in this PR? That would make it easier to test this as a reviewer as well :)
The wiki isn't part of the repo, so I can't share a PR or add the changes. In essence, it would come down to this:
I can also link to the conda website for installation. Anything else you think is missing? |
Also just a reminder since this is a decent change; please wait 24 hours for other folks to give feedback (even tho I think all have chimed in) |
100%. This applies to all PRs (except perhaps changes of the magnitude of a typo fix, etc.). |
457de8e
to
0f75bef
Compare
56c6f73
to
7f76556
Compare
7f76556
to
c9030a1
Compare
Description
This PR is a proposal to remove poetry in favor of just using conda from now on. Note that corresponding changes to the wiki will have to be made after merging (if we do choose to go with this).
Tests
Documentation