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 readthedocs.yaml to new build specification #8383

Merged
merged 1 commit into from
Oct 28, 2021

Conversation

jakevdp
Copy link
Collaborator

@jakevdp jakevdp commented Oct 27, 2021

This change was prompted by the discussion in jupyter/jupyter_core#246

Although jupyter-core is pushing a fix, a more long-term fix was suggested: to follow https://blog.readthedocs.com/new-build-specification/ and update the way we specify our RTD build environment. I also updated our readthedocs environment to Python 3.9, and removed the now-unnecessary pinning of jupyter-core.

@jakevdp
Copy link
Collaborator Author

jakevdp commented Oct 27, 2021

Once jupyter-core 4.9.1 is released I'll repurpose this PR to remove the pinning.

@jakevdp
Copy link
Collaborator Author

jakevdp commented Oct 27, 2021

Second commit is testing this solution: jupyter/jupyter_core#246 (comment)

@jakevdp jakevdp self-assigned this Oct 28, 2021
@jakevdp
Copy link
Collaborator Author

jakevdp commented Oct 28, 2021

Last commit updates to Python 3.9 & uses RTD config from https://blog.readthedocs.com/new-build-specification/

@jakevdp jakevdp changed the title Test pinning of jupyter-core to 4.9.1rc0 Update readthedocs.yaml to new build specification Oct 28, 2021
@google-ml-butler google-ml-butler bot added kokoro:force-run pull ready Ready for copybara import and testing labels Oct 28, 2021
@copybara-service copybara-service bot merged commit 7dba981 into jax-ml:main Oct 28, 2021
@jakevdp jakevdp deleted the test-jupyter-core branch October 28, 2021 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes pull ready Ready for copybara import and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants