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 #163

Merged
merged 5 commits into from
Dec 20, 2024
Merged

ci: add workflow to deploy pkgdown site #163

merged 5 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:

  • register pkgdown site URL for pkgdown linking

  • add the yspec logo

  • generate and track favicons for the site

  • run check_pkgdown as part of the main CI check

The workflow has already been triggered to render https://metrumresearchgroup.github.io/yspec/:

This allows pkgdown to link to this package's site when building other
packages: https://pkgdown.r-lib.org/articles/linking.html
This was generated by calling usethis::use_logo with
<https://merge.metrumrg.com/expo/expo1-nonmem-foce/img/hex/yspec-hex.png>.

I've adjusted the README links output by use_logo to specify a width
consistent with what's used for pmplots and pmtables.
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:19
@kyleam kyleam merged commit 9db097c into main Dec 20, 2024
11 checks passed
@kyleam kyleam deleted the actions-site branch December 20, 2024 15:36
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