-
Notifications
You must be signed in to change notification settings - Fork 119
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
[develop]: Update doc requirements and add logo #1018
[develop]: Update doc requirements and add logo #1018
Conversation
…p into text/us-25
…er-app into text/rtd-update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gspetro-NOAA - These changes look good to me! The new logo in RTD looks good and the modification to the background color really makes it pop!
It looks like there is a conflict with the ConfigWorkflow.rst
following the latest merge. At your earliest convenience, please merge the latest HEAD into your text/rtd-update
branch and I will be able to merge this back to develop. Thanks!
DESCRIPTION OF CHANGES:
requirements.in
requirements.txt
based onrequirements.in
for a full list of dependencies. This will ensure uniform documentation builds across platforms.docs
directory todoc
for NCO compliance.NOTE: Most file changes are just a rename. Changes that need reviewing appear in:
conf.py
,requirements.txt
,conf.py
,.readthedocs.yaml
, andtheme_overrides.css
Type of change
TESTS CONDUCTED:
None required. Documentation for PR can be viewed at the bottom of the PR.
DEPENDENCIES:
Merge PR #1014 first.
DOCUMENTATION:
All documentation.
ISSUE:
Resolves Issue #1017 .
CHECKLIST
CONTRIBUTORS (optional):