Fix workflow dependency installation - #5
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - name: Install dependencies | ||
| shell: Rscript {0} | ||
| run: | | ||
| install.packages(c( | ||
| "testthat", "tidyverse", "rstatix", "ggstatsplot", "ggsignif", | ||
| "see", "ggpmisc", "writexl", "readxl", "emoa", "effectsize", | ||
| "easystats" |
There was a problem hiding this comment.
Install curl before running tests
The workflow step at lines 15‑21 only installs a handful of CRAN packages before running the suite, but the helper sets options(rcode.auto_install = FALSE) so tests rely entirely on the packages preinstalled here. The new basic utility helpers behave test (tests/testthat/test_r_functionality.R:97‑99) stubs curl::has_internet via with_mock. On a clean runner where the curl package isn’t already available (it’s not a recommended/base package), referencing curl::has_internet fails before the mock is applied, so the tests abort and the workflow can never turn green. Please add curl (and any other packages referenced directly in the tests) to this install.packages list so CI reliably succeeds.
Useful? React with 👍 / 👎.
Summary
install.packages()executes successfullyeasystatsis preinstalled alongside the other test dependencies so sourcingr_functionality.Rno longer failsTesting
Codex Task