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

Introduce a GitHub Actions workflow for publishing the website #16364

Merged
merged 1 commit into from
Apr 27, 2023

Conversation

timvandermeij
Copy link
Contributor

@timvandermeij timvandermeij commented Apr 27, 2023

This commit migrates this functionality away from the bots. Nowadays it's possible to build and deploy the website to GitHub Pages directly through the GitHub Actions, which provides a nice simplication of the process. Not only does this remove the requirement to have a gh-pages branch in the repository, it also avoids the complexity of having to configure the workflow to commit to Git branches and allows us to remove the Git committing code from the Gulpfile.

Note that deploying directly though GitHub Actions workflows needs to be enabled in the repository settings, but this is easy and well documented on the link below.

The following resources are relevant for this patch:

You can view this workflow in action on my fork at https://github.com/timvandermeij/pdf.js/actions/runs/4822609834 where I've done nothing other than enabling GitHub Pages deployment from GitHub Actions workflows in the settings of the fork. The published website can be viewed at https://timvandermeij.github.io/pdf.js.

Implements the first step of #11851 to move the non-testing responsibilities away from the bots.
Relates to mozilla/botio-files-pdfjs#39.

Required actions once this PR is approved:

This commit migrates this functionality away from the bots. Nowadays
it's possible to build and deploy the website to GitHub Pages directly
through the GitHub Actions, which provides a nice simplication of the
process. Not only does this remove the requirement to have a `gh-pages`
branch in the repository, it also avoids the complexity of having to
configure the workflow to commit to Git branches and allows us to remove
the Git committing code from the Gulpfile.

Note that deploying directly though GitHub Actions workflows needs to be
enabled in the repository settings, but this is easy and well documented
on the link below.

The following resources are relevant for this patch:

- Enabling deployment to GitHub Pages directly through GitHub Actions:
  https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site#publishing-with-a-custom-github-actions-workflow
- Uploading GitHub Pages artifacts example:
  https://github.com/actions/upload-pages-artifact#usage
- Deploying GitHub Pages artifacts example:
  https://github.com/actions/deploy-pages#usage
Copy link
Collaborator

@Snuffleupagus Snuffleupagus left a comment

Choose a reason for hiding this comment

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

r=me, this is really nice; thank you!


@calixteman Can you please take care of the steps outlined in #16364 (comment)?

@marco-c marco-c merged commit 9eea999 into mozilla:master Apr 27, 2023
@marco-c marco-c assigned marco-c and unassigned calixteman Apr 27, 2023
@marco-c
Copy link
Contributor

marco-c commented Apr 27, 2023

All done, except the last step that is for you @timvandermeij !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants