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 release workflow not publishing latest tags for docs #277

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tomvothecoder
Copy link
Collaborator

@tomvothecoder tomvothecoder commented Jun 21, 2023

I noticed the E3SM Diags release workflow wasn't copying the docs for v2.8.0 over to the gh-pages branch (related PR). This caused the docs to break with a 404 if you tried clicking v2.8.0 in the dropdown.

The same thing is happening in zstash with v1.3.0. I updated the release workflow to correctly copy the tag docs. Once this is merged, we can rerun the release workflow manually to copy v1.3.0 docs over to gh-pages.

Note, another 1.4.0 release candidate is not required for this fix because this is DevOps related.

@tomvothecoder tomvothecoder requested a review from forsyth2 June 21, 2023 22:14
@@ -4,6 +4,8 @@ on:
release:
types: [published]

workflow_dispatch:
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line allows us to manually run workflows. We need to add this because the release workflow currently only triggers when releasing new versions. However, we want to run it manually to fix the v1.3.0 docs.

@tomvothecoder tomvothecoder changed the title Fix release workflow not publishing latest tags Fix release workflow not publishing latest tags for docs. Jun 21, 2023
@tomvothecoder tomvothecoder changed the title Fix release workflow not publishing latest tags for docs. Fix release workflow not publishing latest tags for docs Jun 21, 2023
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.

1 participant