Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix release doc build, update changelog for 7.0.3 release. #568

Merged
merged 5 commits into from
Apr 3, 2023

Conversation

mdickinson
Copy link
Member

@mdickinson mdickinson commented Apr 3, 2023

This PR fixes the automated release documentation build.

@mdickinson
Copy link
Member Author

Okay, this now seems to be working / failing in the expected manner. We end up executing

python docs/update_gh_pages.py docs/build/html ../docs --tag 568/merge

and then failing (as expected) because 568/merge doesn't have the expected form.

@mdickinson mdickinson marked this pull request as ready for review April 3, 2023 15:36
@@ -27,7 +27,7 @@ jobs:
git worktree add ../docs gh-pages
- name: Update the docs on the gh-pages branch
run: |
python docs/update_gh_pages.py docs/build/html ../docs --tag ${{ GITHUB_REF_NAME }}
python docs/update_gh_pages.py docs/build/html ../docs --tag ${{ github.ref_name }}
Copy link
Member Author

Choose a reason for hiding this comment

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

This was a really dumb mistake. I think ${{ env.GITHUB_REF_NAME }} would also work here, but this seems to work in testing.

CHANGES.rst Outdated Show resolved Hide resolved
@mdickinson
Copy link
Member Author

@Supianti0115 Why are you reviewing something that was merged over a year ago?

@mdickinson mdickinson changed the title Fix release doc build Fix release doc build, update changelog for 7.0.3 release. Sep 5, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants