-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
push a new version to CRAN? #153
Comments
Hi, @bbolker. For my contributions (mainly unit tests but some error fixes and efficiency tweaks), the package passed -as-CRAN. I'm pretty sure it did for @eddelbuettel as well, but it's up to @astamm to decide :). |
Hi guys. Thanks for the reminder. I am on it. |
Cough, cough, nudge, poke, wink, smile, cough… |
Still hoping for this (although I certainly have my own problems to deal with as well) |
@astamm, what could we do to help you prepare for a new release? Is there anything outstanding in your mind or is it simply a matter of time? |
Hi, @bbolker. I just sent PR #157 to @astamm, which addresses three outstanding issues. I do have the idea to finish cleaning up the examples and adding some more unit tests for accuracy (based on those examples), but I am not sure if that's worth holding up a new CRAN release, being that it has been so long. See what's under 2.1.0 in the PR for just how much has changed! |
I apologise for the limited bandwidth. I will submit a new release to CRAN next week. |
On its way to CRAN as of today. |
97.53% coverage, so beautiful! |
Thanks to you @aadler ! |
The version on CRAN is two years old, and there are useful things in the development version. (I'm particularly interested in the switch to
tinytest
and corresponding decrease in upstream dependencies ...)I haven't tried running
R CMD check
, but if you do encounter any submission-related issues, I would be happy to help troubleshoot.The text was updated successfully, but these errors were encountered: