Skip to content
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

Update docs readme #386

Merged
merged 1 commit into from
Aug 31, 2016
Merged

Update docs readme #386

merged 1 commit into from
Aug 31, 2016

Conversation

mpacer
Copy link
Member

@mpacer mpacer commented Aug 31, 2016

Pursuant to #379, this seemed like a good thing to make as a quick fix, with the more substantial restructuring to occur later.

Notably this requires using conda, which not all users may rely on. I do not know if there is a way to expose the environment.yml file to a pip command for non-conda users, but it seems like it shouldn't be a long run requirement that you use conda in order to build our docs.

cc @willingc

Pursuant to #379, this seemed like a good thing to make as a quick fix, with the more substantial restructuring to occur later. 

Notably this requires using conda, which not all users may rely on. I do not know if there is a way to expose the `environment.yml` file to a `pip` command for non-conda users, but it seems like it shouldn't be a long run requirement that you use `conda` in order to build our docs.
@willingc willingc merged commit 9132715 into master Aug 31, 2016
@willingc
Copy link
Member

@minrk FYI. This Travis build has one of the intermittent kernel dying messages during one of the tests. https://travis-ci.org/jupyter/nbconvert/jobs/156625723

@michaelpacer Thanks for doing this. I agree that it's worthwhile to add instructions on building locally with pip too. I'm going to merge this now, and we can follow up with pip in a future PR. ☀️

@minrk minrk deleted the michaelpacer-docs-readme branch September 2, 2016 12:15
@minrk minrk added this to the 5.0 milestone Sep 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants