Closed infotroph closed 6 months ago
Test failures indicate a thinko in my test approach: Assumes PEcAn.all is already installed, so works during package check (R CMD check installs it to a temp library) but not when testing with testthat
before installation (which loads the code into an R session but doesn’t change the output from packageVersion
).
Marking this as draft while I think more; suggestions welcome in the meantime
Description
Uses the sources reported by
sessioninfo::package_info
; ifsessioninfo
isn't available* it falls back to the current behavior of only reporting versions.Also adds a call to
pecan_version
to the default workflow, so that package versions get recorded in the run log.Output on my machine before this change (and after it if sessioninfo weren't installed):
Output on my machine after this change:
*I think most PEcAn machines can be assumed to have
sessioninfo
; it's required bydevtools
. But this way we avoid a hard dependency.Motivation and Context
Review Time Estimate
Types of changes
Checklist: