Skip to content

run extra checks for cran prep#5

Merged
malcolmbarrett merged 1 commit into
mainfrom
extra-checks
Mar 21, 2026
Merged

run extra checks for cran prep#5
malcolmbarrett merged 1 commit into
mainfrom
extra-checks

Conversation

@malcolmbarrett

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR appears to be CRAN-release preparation for the causalpie R package, focusing on documentation/package metadata updates rather than functional changes.

Changes:

  • Added return-value documentation for the theme helpers (roxygen + generated Rd).
  • Updated installation instructions to include CRAN install and switched GitHub install guidance to pak.
  • Added standard CRAN-release housekeeping (NEWS, copyright holder role, ignore cran-comments).

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
man/themes.Rd Documents return value for theme functions in generated Rd.
R/causalpie.R Adds @return roxygen for theme helpers (drives Rd update).
README.md Adds CRAN install snippet; updates dev install to pak.
README.Rmd Same as README.md (source for generated README).
NEWS.md Adds initial release notes for 0.1.0.
DESCRIPTION Adds cph role; removes redundant Maintainer field.
LICENSE / LICENSE.md Updates copyright year metadata/text.
.gitignore / .Rbuildignore Ignores cran-comments.md (git + build).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@malcolmbarrett malcolmbarrett merged commit 55f572d into main Mar 21, 2026
10 checks passed
@malcolmbarrett malcolmbarrett deleted the extra-checks branch March 21, 2026 21:31
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