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.yml configuration #1613

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

marshallward
Copy link
Collaborator

Newest Read the Docs configuration file requires explicit specification of the environment (using build:). This patch includes this section.

Upgrading to newer Python environments has also forced us to specify an older Jinja2 that works with our legacy Sphinx module.

Newest Read the Docs configuration file requires explicit specification
of the environment (using `build:`).  This patch includes this section.

Upgrading to newer Python environments has also forced us to specify an
older Jinja2 that works with our legacy Sphinx module.
@marshallward
Copy link
Collaborator Author

This is a very minor PR which only modifies Read The Docs configuration. Currently, our documentation generation does not work and gives an error.

Although documentation is produced from the NOAA-GFDL fork, it is generated from our copy of main, so we need this to go through the MOM-ocean fork before we can update our own copy.

This should not affect the others in any way and should be quick to approve.

@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #1613 (9ea8074) into main (f6b6b0b) will not change coverage.
The diff coverage is n/a.

❗ Current head 9ea8074 differs from pull request most recent head e41929e. Consider uploading reports for the commit e41929e to get more accurate results

@@           Coverage Diff           @@
##             main    #1613   +/-   ##
=======================================
  Coverage   37.91%   37.91%           
=======================================
  Files         269      269           
  Lines       77155    77155           
  Branches    14164    14164           
=======================================
  Hits        29254    29254           
  Misses      42623    42623           
  Partials     5278     5278           

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@marshallward
Copy link
Collaborator Author

Reminder to have a look at this PR. It is only to fix a minor documentation issue.

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.

6 participants