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

Fix 'readthedocs' #861

Merged

Conversation

GeorgeGayno-NOAA
Copy link
Collaborator

@GeorgeGayno-NOAA GeorgeGayno-NOAA commented Oct 17, 2023

DESCRIPTION OF CHANGES:

  • Add a new .readthedocs.yaml file.
  • Remove references to sphinxconfig.bibtex, which is not currently used by our documentation.

TESTS CONDUCTED:

All done using the branch at ed637cc.

  • Compile branch on all Tier 1 machines using Intel (Orion, Jet, Hera and WCOSS2).
  • Compile branch on Hera using GNU.
  • Compile branch in 'Debug' mode on WCOSS2.
  • Run unit tests locally on any Tier 1 machine. Done on Hera.

Consistency tests were not run as no source code was changed.

DEPENDENCIES:

None.

DOCUMENTATION:

ReadTheDocs was successfully built: https://georgegayno-noaaufs-utils.readthedocs.io/en/bugfix-bnu_soil/

Doxygen was built with no issues.

ISSUE:

Fixes #860

CONTRIBUTORS:

With thanks to @gspetro-NOAA

@GeorgeGayno-NOAA
Copy link
Collaborator Author

@gspetro-NOAA - your review is welcome.

@GeorgeGayno-NOAA GeorgeGayno-NOAA merged commit b31385f into ufs-community:develop Oct 18, 2023
4 checks passed
@GeorgeGayno-NOAA GeorgeGayno-NOAA deleted the bugfix/readthedocs branch October 30, 2023 15:17
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.

'readthedocs' build is failing
2 participants