-
Notifications
You must be signed in to change notification settings - Fork 4
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
Review RC [doc]: environment instructions for dev #63
Comments
cf #64 The instructions should probably be:
|
Hi @rcaneill - This (pip install from requirements_dev) was added to the docs contributing page on the pyOpenSci-review branch. Once that is passing, merged and the docs built does it seem adequate to close this issue? |
It's good to see it somewhere. However, I believe that "explicit is better than implicit", and so providing the exact commands to type should be done, as in #63 (comment) I can open a PR for that, or let you do it (or if you disagree please tell me why) |
Hi,
I am starting my in-depth review and will open issues with my questions / comments on the fly.
I believe that the package is missing some instructions for devs, like:
Here is what I did, I guess it is what should be done:
The text was updated successfully, but these errors were encountered: