-
Notifications
You must be signed in to change notification settings - Fork 341
ctsm5.3.048: Automatically publish docs to this repo #3146
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
Conversation
# Conflicts: # .github/workflows/docker-image-build-publish.yml # .github/workflows/docker-image-build.yml # .github/workflows/docker-image-common.yml # .github/workflows/docs-ctsm_pylib.yml # .github/workflows/docs.yml # .gitignore # .gitmodules # doc/ChangeLog # doc/ChangeSum # doc/ctsm-docs_container/README.md # doc/source/conf.py # doc/source/users_guide/working-with-documentation/building-docs-original-wiki.md # doc/source/users_guide/working-with-documentation/docs-intro-and-recommended.md # doc/source/users_guide/working-with-documentation/index.rst
# Conflicts: # .github/workflows/docker-image-build.yml # .github/workflows/docker-image-common.yml # .github/workflows/docs-ctsm_pylib.yml # .github/workflows/docs.yml # .github/workflows/fleximod_test.yaml # .github/workflows/formatting_python.yml # doc/source/conf.py
Only run on push to master, and only if certain files are updated.
ekluzek
left a comment
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.
@samsrabin and I went over this. And this looks great. We talked about a few things to do. But, approving so he doesn't have to wait for me.
This also will be rebased to deploy on master rather than b4b-dev. There are extra diffs shown here because there are changes from master that show up here. But, that will go away, once b4b-dev goes to master, and this is rebased to master.
Failing 🤷. Trust that doc-builder does this test.
Merge b4b-dev to master
Description of changes
Specific notes
Contributors other than yourself, if any: None
CTSM Issues Fixed:
Are answers expected to change (and if so in what way)? No
Any User Interface Changes (namelist or namelist defaults changes)? Not to the model, no.
Does this create a need to change or add documentation? Did you do so? Yes; yes.
Testing performed, if any: See checks.
Remaining work
ctsm-docsrepoinclude::ed files as triggers for docs workflowsbuild-docs-omnibus-testis expected to fail untilversions_list.pyis updated to point to clm5.0 release branchWaiting on b4b-dev merge
After merging
testing.shand breaking its workflow up into different jobs.