-
Notifications
You must be signed in to change notification settings - Fork 4
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
Prepare for CRAN release #149
Comments
Hi @Bisaloo, thanks for your review 😊:
|
5 tasks
Hi Hugo, The package is now avaliable on CRAN and the items mentioned above have been resolved. Huge thanks to all of you for your collaboration and support in this journey 😊🚀🙏. I'll proceed to close this issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have not reviewed the package code or design but upon a cursory look, here are the points that almost surely will cause issues with CRAN:
data
andinst/extdata
, can this be fixed?getting_started.Rmd
andsivirep.Rmd
vignettes seem mostly duplicated (both leading to an almost 50MB HTML file)cran-comments.md
to scope of the package, why it has to be in Spanish, and how you have tried to follow recommendation from the official 'Writing R Extensions' guide: https://cran.r-project.org/doc/manuals/R-exts.html#The-DESCRIPTION-file-1The text was updated successfully, but these errors were encountered: