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 w/newly required fields #805

Merged
merged 5 commits into from
Oct 19, 2023

Conversation

gspetro-NOAA
Copy link
Collaborator

This PR adds newly required fields to the .readthedocs.yml file.
The addition of the sphinx_rtd_theme to the list of extensions in conf.py and the addition of a requirements file were also necessary. Docs successfully build on my fork: https://gsp-upp.readthedocs.io/en/text-rtd-update/
This will resolve Issue #803 .

@WenMeng-NOAA WenMeng-NOAA linked an issue Oct 18, 2023 that may be closed by this pull request
@gspetro-NOAA gspetro-NOAA added the bug Something isn't working label Oct 18, 2023
@WenMeng-NOAA WenMeng-NOAA added Ready for Review This PR is ready for code review. documentation Improvements or additions to documentation labels Oct 18, 2023
@WenMeng-NOAA
Copy link
Collaborator

@gspetro-NOAA Can you update your branch so we can process this PR?

@WenMeng-NOAA WenMeng-NOAA added the No Baseline Change No baseline of the UPP regression tests are made. label Oct 19, 2023
@WenMeng-NOAA
Copy link
Collaborator

The UPP RTs were completed on WCOSS2 without changes. This PR is ready for merging.

@WenMeng-NOAA WenMeng-NOAA merged commit fa09239 into NOAA-EMC:develop Oct 19, 2023
@gspetro-NOAA gspetro-NOAA deleted the text/rtd-update branch January 23, 2024 17:09
@gspetro-NOAA gspetro-NOAA restored the text/rtd-update branch January 23, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation No Baseline Change No baseline of the UPP regression tests are made. Ready for Review This PR is ready for code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Builds are failing on Read The Docs
2 participants