Add effect sizes, tests, and CI workflow #4
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
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 | ||
| run: | | ||
| install.packages(c( | ||
| "testthat", "tidyverse", "rstatix", "ggstatsplot", "ggsignif", | ||
| "see", "ggpmisc", "writexl", "readxl", "emoa", "effectsize" | ||
| )) |
There was a problem hiding this comment.
The workflow step that is supposed to install the R packages (lines 15‑20) runs install.packages(...) as a plain shell command. GitHub Actions steps default to bash, so the runner throws install.packages: command not found before any dependencies are installed or tests run. Wrap the installation in an R invocation (e.g. Rscript -e 'install.packages(...)' or shell: Rscript {0}) so the command executes inside R and the job can proceed.
Useful? React with 👍 / 👎.
| install.packages(c( | ||
| "testthat", "tidyverse", "rstatix", "ggstatsplot", "ggsignif", | ||
| "see", "ggpmisc", "writexl", "readxl", "emoa", "effectsize" | ||
| )) |
There was a problem hiding this comment.
In the helper (tests/testthat/helper-r_code.R:2) the tests disable automatic installation, yet r_functionality.R still executes library(easystats) (line 57). The workflow’s dependency list (lines 17‑20) never installs easystats, so sourcing the file immediately errors with “there is no package called ‘easystats’” and the suite never runs. Ensure that easystats (and any other packages loaded unconditionally) are installed in this step before sourcing the script, or stop disabling auto installs for the workflow.
Useful? React with 👍 / 👎.
Summary
testthatsuite covering every helper plus supporting helpers for sourcingTesting
Codex Task