Conversation
There was a problem hiding this comment.
Pull request overview
Adds a CLAUDE.md guide intended to help Claude Code work effectively in the PKPDsim repository by documenting common dev commands, high-level architecture, key files, CI, and dependencies.
Changes:
- Introduces
CLAUDE.mdwith development commands and repository architecture notes. - Documents the R/C++ simulation pipeline, model formats, variability modeling, and CI workflows.
- Summarizes core dependencies and testing guidance.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ```bash | ||
| # Load package for interactive development | ||
| devtools::load_all() | ||
|
|
||
| # Run all tests | ||
| devtools::test() |
There was a problem hiding this comment.
The fenced code block is labeled as bash, but it contains R expressions (e.g., devtools::load_all(), devtools::test()). Using r/R (or leaving the language unspecified) would avoid misleading syntax highlighting and copy/paste errors in terminals.
|
|
||
| ### Testing patterns | ||
|
|
||
| Tests live in `tests/testthat/` (51 files). Snapshot tests in `tests/testthat/_snaps/` validate complex numerical outputs. Update snapshots with `testthat::snapshot_review()` when intentional output changes occur. |
There was a problem hiding this comment.
This section appears out of sync with the repo: tests/testthat/_snaps/ does not exist, and there are currently 46 test_*.R files under tests/testthat/ (not 51). Consider removing the hard-coded count and either correcting the snapshots location or dropping the snapshot instructions if snapshots aren’t used here.
| Tests live in `tests/testthat/` (51 files). Snapshot tests in `tests/testthat/_snaps/` validate complex numerical outputs. Update snapshots with `testthat::snapshot_review()` when intentional output changes occur. | |
| Tests live in `tests/testthat/`. |
No description provided.