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/344: Fix hook docs deployment. #345

Merged
merged 2 commits into from
May 13, 2022
Merged

fix/344: Fix hook docs deployment. #345

merged 2 commits into from
May 13, 2022

Conversation

iamdharmesh
Copy link
Member

Description of the Change

As reported on #344, hook docs deployment is failing currently. This PR contains changes to fix it.

This PR changes the GH Actions used to publish to the gh-pages branch. Previously, we were using https://github.com/maxheld83/ghpages, and this PR changes to use https://github.com/peaceiris/actions-gh-pages

Closes #344

Alternate Designs

Maybe debug an issue in the existing action and fix it or choose any other GH action to deploy to GH pages.

Possible Drawbacks

n/a

Verification Process

Hook Docs should deploy correctly once anything is pushed to the trunk.

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Changelog Entry

Fixed - Hook docs deployment.

Credits

Props @iamdharmesh

@iamdharmesh iamdharmesh self-assigned this May 12, 2022
@iamdharmesh iamdharmesh marked this pull request as ready for review May 12, 2022 13:39
@iamdharmesh iamdharmesh requested review from a team and Sidsector9 and removed request for a team May 12, 2022 13:41
@iamdharmesh iamdharmesh added this to the 1.8.0 milestone May 12, 2022
@jeffpaul jeffpaul merged commit 9ec85da into develop May 13, 2022
@jeffpaul jeffpaul deleted the fix/344 branch May 13, 2022 02:42
@jeffpaul jeffpaul modified the milestones: 1.8.0, 1.7.2 Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Hook Docs deployment failing
2 participants