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

ci: add workflow to deploy pkgdown site #349

Merged
merged 7 commits into from
Dec 20, 2024
Merged

ci: add workflow to deploy pkgdown site #349

merged 7 commits into from
Dec 20, 2024

Conversation

kyleam
Copy link
Contributor

@kyleam kyleam commented Dec 19, 2024

This PR wires up GitHub Actions to publish the pkgdown site via gh-pages on push of a release tag. (The deploy can also be triggered manually or by push of a specific branch, scratch/deploy-site.)

In addition to adding the workflow, this PR makes several related changes:

  • add pkgdown configuration

  • ignore the docs/ directory

  • generate and track favicons for the site

  • fix links in README

  • run check_pkgdown as part of the main CI check

The gh-pages branch has been created, but the deployment hasn't been triggered yet because the repo settings need to be adjusted to deploy from the gh-pages branch.

This follows pmplots in using the default theme.
Use the more typical location (which is expected by
pkgdown::build_favicons).
Generated with pkgdown::build_favicons()

Note that these aren't compatible with the current pkgdown release
(v2.1.1) due to a breaking change in
<https://realfavicongenerator.net/>.  This issue is fixed with
pkgdown's dd777ee8 (Update favicon links in head ..., 2024-11-26).
This will help catch pkgdown issues when they are introduced rather
than when the site is built on release.
@kyleam kyleam requested a review from kylebaron December 19, 2024 19:31
@kyleam kyleam merged commit 6473fea into main Dec 20, 2024
11 checks passed
@kyleam kyleam deleted the actions-site branch December 20, 2024 15:42
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