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

Migrate GitHub Pages deployment away from the static-websites action #231

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

miikka
Copy link

@miikka miikka commented Oct 10, 2024

Background: rust-lang/simpleinfra#597

This PR migrates the deployment to GitHub pages away from the custom static-websites GHA action to the official actinos/upload-github-pages-artifact and actions/deploy-pages actions.

For this to work, this repo needs to be configured to use GitHub Actions as the source for Pages.

I tested this in my own fork and it seemed to work (with the if: conditions commented out as they would prevent the interesting parts from running).

@MarcoIeni
Copy link
Member

MarcoIeni commented Oct 10, 2024

As suggested from Miikka we need to change the settings of the repo immediately before merge the PR
image

@MarcoIeni
Copy link
Member

MarcoIeni commented Oct 11, 2024

Looks good to me, thanks 🙌
Let's wait for the maintainers of the repo to approve it 👍
After that, I can change the settings of the repo and merge the PR 🚀

@MarcoIeni
Copy link
Member

Hey @nrc, can the infra team take care of this?
You don't need to review the PR, we'll merge it and make sure that the async book website still works and follow the latest best practices 👍

@nrc
Copy link
Member

nrc commented Dec 3, 2024

Hey @nrc, can the infra team take care of this? You don't need to review the PR, we'll merge it and make sure that the async book website still works and follow the latest best practices 👍

That would be great, thank you!

@MarcoIeni MarcoIeni self-assigned this Dec 3, 2024
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.

3 participants