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

Updated GitHub workflow to upload documentation to web site #965

Merged
merged 4 commits into from
Aug 31, 2023

Conversation

swryan
Copy link
Contributor

@swryan swryan commented Aug 15, 2023

Summary

Updated GitHub workflow to upload documentation to the web site, where a version will be kept for each release going forward.

Related Issues

Backwards incompatibilities

None

New Dependencies

None

@coveralls
Copy link

coveralls commented Aug 15, 2023

Coverage Status

coverage: 92.571% (+0.002%) from 92.569% when pulling 2f849ed on swryan:916_upload_docs into 1aca42a on OpenMDAO:master.

@swryan swryan marked this pull request as draft August 15, 2023 19:01
@swryan swryan marked this pull request as ready for review August 16, 2023 17:06
@swryan
Copy link
Contributor Author

swryan commented Aug 16, 2023

Because the doc build uses the latest release version of OpenMDAO, this should not be merged until a new release of OpenMDAO has been made that includes PR #2994.

We could temporarily install the latest dev version of openmdao for the upload step (which was done for testing), but since an OpenMDAO release is imminent, I suggest we just hold off on merging this PR until afterwards.

@swryan swryan marked this pull request as draft August 16, 2023 19:03
@swryan swryan marked this pull request as ready for review August 30, 2023 18:24
@swryan
Copy link
Contributor Author

swryan commented Aug 30, 2023

OpenMDAO 3.28.0 has been released with the fix to the doc upload utility, so this should be good to go now.

@robfalck robfalck merged commit 42d9275 into OpenMDAO:master Aug 31, 2023
8 checks passed
@swryan swryan deleted the 916_upload_docs branch November 2, 2023 17:37
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.

Maintain versions of documentation.
3 participants