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

add pip to docs envt #1571

Merged
merged 1 commit into from
May 14, 2021
Merged

add pip to docs envt #1571

merged 1 commit into from
May 14, 2021

Conversation

lwasser
Copy link
Contributor

@lwasser lwasser commented May 13, 2021

This should address the issue where CI hangs when building docs due to not being able to find pip correctly - note that the this builds upon issues discovered in #1567

@lwasser
Copy link
Contributor Author

lwasser commented May 13, 2021

ok @yuvipanda so this pip fix makes the docs build fast! yay!

Warning, treated as error:
/root/checkout/nbconvert/exporters/exporter.py:docstring of nbconvert.exporters.exporter.Exporter.register_preprocessor:8:more than one target found for 'any' cross-reference 'Preprocessor': could be :py:class:`nbconvert.preprocessors.Preprocessor` or :py:class:`nbconvert.preprocessors.base.Preprocessor`

But this docstring issue is still present. i am not working tomorrow so i won't respond again until next monday. Feel free to edit this PR or just let me k now how to adjust that docstring and i can address it this weekend or monday! :)

@yuvipanda
Copy link
Contributor

This is already an improvement, @lwasser - without this there was hundreds of lines of output in the docs ci :) So I'm going to merge this now, and we can take another stab at the remaining issue when you come back :) Have a good weekend!

@yuvipanda yuvipanda merged commit fa3ba74 into jupyter:master May 14, 2021
@lwasser
Copy link
Contributor Author

lwasser commented May 17, 2021

excellent! glad it was helpful. i'll dig a bit more into that docstring item one evening this week!

@yuvipanda
Copy link
Contributor

Thanks, @lwasser :)

@MSeal MSeal added this to the 6.1 milestone Jun 8, 2021
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