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

Merged
merged 6 commits into from
Dec 20, 2024
Merged

ci: add workflow to deploy pkgdown site #46

merged 6 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

  • untrack and ignore the docs/ directory

  • register pkgdown site URL for pkgdown linking

  • fix a link in the README

  • run check_pkgdown as part of the main CI check

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

news.md should be NEWS.md, but instead update it to a link that will
also work for when the README is rendered as part of the pkgdown site.

Fixes #36
pkgdown requires listing the pkgdown site in the DESCRIPTION's URL in
order to be able to link to this package from another pkgdown site:
https://pkgdown.r-lib.org/articles/linking.html#across-packages

While at it, also add the typical repo URLs.
This follows pmplots in using the default theme.
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:30
@kyleam kyleam merged commit 7db8c95 into main Dec 20, 2024
7 checks passed
@kyleam kyleam deleted the actions-site branch December 20, 2024 15:35
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