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

Dev - fixes RTD deployment #166

Merged
merged 16 commits into from
Aug 20, 2024
Merged

Dev - fixes RTD deployment #166

merged 16 commits into from
Aug 20, 2024

Conversation

pluflou
Copy link
Collaborator

@pluflou pluflou commented Aug 19, 2024

This PR addresses the workflow that deploys and updates the read the docs pages. It retriggers the webhook to update the version number post-release. Note that the re-deployment of the docs should only trigger at PR with main and at release. Nevermind, I removed the changes to the build files and just updated the webhook instead to trigger at pushes, PR, and release. This PR just cleans up the way RTD builds.

Checklist for PR authors (skip items if you don't have permissions or they are not applicable)

  • Updated or added relevant tests -- N/A
  • Updated relevant documentation -- N/A
  • Added relevant label(s) -- N/A
  • All comments are resolved

@pluflou
Copy link
Collaborator Author

pluflou commented Aug 19, 2024

@bmeyers you can push your changes to the docs to this branch if you want

@pluflou pluflou temporarily deployed to test-deployment August 19, 2024 21:11 — with GitHub Actions Inactive
@pluflou pluflou temporarily deployed to test-deployment August 19, 2024 21:31 — with GitHub Actions Inactive
@bmeyers
Copy link
Collaborator

bmeyers commented Aug 19, 2024

@bmeyers you can push your changes to the docs to this branch if you want

Sounds good!

@bmeyers bmeyers temporarily deployed to test-deployment August 19, 2024 22:35 — with GitHub Actions Inactive
@pluflou pluflou merged commit 5751521 into main Aug 20, 2024
5 checks passed
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.

2 participants