Closes #245 - Replace scattered install.packages() with DESCRIPTION + r-lib/actions/setup-r-dependencies#379
Draft
Closes #245 - Replace scattered install.packages() with DESCRIPTION + r-lib/actions/setup-r-dependencies#379
Conversation
…tup-r-dependencies Agent-Logs-Url: https://github.com/pharmaverse/blog/sessions/00d31ec3-dfeb-4bff-a5e4-dafb77daff7f Co-authored-by: bms63 <10111024+bms63@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add package check and installation in create_blogpost
Closes #[issue] - Replace scattered install.packages() with DESCRIPTION + r-lib/actions/setup-r-dependencies
May 4, 2026
Contributor
|
👀 Preview available! The preview website for this PR is available at: The preview is updated every time a new commit is pushed to this PR. |
Collaborator
|
@aaehmann and @ashley-tarasiewicz this is a QoL fix to help folks easily install everything in one go. Do you mind taking a look and seeing if it works - like create a mock blog post. This will probably be my last PR - hoping to hand off the blog officially to you all with minimal long-term fuss!! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Package dependencies were maintained as duplicate hardcoded
install.packages()lists across workflows, the README, and the Dockerfile — with no single source of truth and no caching.Changes
DESCRIPTION(new file)Dummy package descriptor (not a real package — same pattern as
pharmaverse/examples) declaring all blog site dependencies as the canonical single source of truth. Scanned all posts forlibrary()calls to ensure completeness. Key packages included:admiral*family,pharmaverse*data packagesteal/tern/rtables/rlistingsecosystemmirai,dverse,autoslider.core,sdtm.oak,filters,link,riskmetric, etc.All 6 workflows updated
Replaced
container: image: rocker/tidyverse/ghcr.io/pharmaverse/docker_pharmaverse+ manualinstall.packages()with:Benefits: PPM binary installs (fast), built-in caching (no re-downloads unless new version), zero per-workflow package lists to maintain.
Lightweight workflows (
spellcheck,style_check,check_post_tags,link_check) use targetedpackages:input to avoid installing the full dependency graph.README.mdReplaced 30-line
install.packages()block with:R/create_blogpost.RAdded commented
pak::pak()hint so new contributors know how to bootstrap their environment.Thank you for your Pull Request! We have developed this task checklist to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your blog post.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the
mainbranch until you have checked off each task."posts/date_with_short_tile"and the blog post file and any other artifacts have informative but succinct names. (N/A — no new post)CICD.Rline by line to first check the spelling in your post and then to make sure your code is compatible with our code-style. Address any incongruences by following the instructions in the file! (N/A — no new post)tag(s)or categories from the current list:c("Metadata", "SDTM", "ADaM", "TLG", "Shiny", "Python", "Community", "Conferences", "Submissions", "Technical", "DEI")for your blog post. (N/A — no new post)descriptionfield at the top of the markdown document. (N/A — no new post)Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
cran.r-project.org/usr/bin/curl curl -s REDACTED(dns block)crandb.r-pkg.org/usr/bin/curl curl -s REDACTED(dns block)If you need me to access, download, or install something from one of these locations, you can either: