chjackson / flexsurv

The flexsurv R package for flexible parametric survival and multi-state modelling
http://chjackson.github.io/flexsurv/
53 stars 28 forks source link

Fix p-value issue with tidy() and add unit test #160

Closed mattwarkentin closed 1 year ago

mattwarkentin commented 1 year ago

Fixes #159. Thanks to @huftis for catching the bug!

mattwarkentin commented 1 year ago

@chjackson devtools::check() failed on my local machine for reasons seemingly unrelated to this PR, but just wanted to point that out. Package was able to be built successfully and all tests passed.

chjackson commented 1 year ago

Thanks Matt , it just needed broom to be added to Suggests: if you use broom::tidy in the package tests. I'll add this.