-
Notifications
You must be signed in to change notification settings - Fork 130
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
Submit version 1.0 to CRAN #140
Comments
Getting closer... |
I think this is a very worthy 1.0 release! I was just showing off some of the new functionality today to someone and couldn't work out why it wasn't working (they had the CRAN release installed). My suggestion from your pending list, FWIW, is to
Full disclosure: I want to use it in my teaching for R novices without having to introduce |
I really appreciate that, Garth. I've been haunted by this idea that if I'm calling it 1.0, I should get everything just right. But I think the functionality is finished, and at this point I'm hurting the user experience by requiring installation from GitHub. Too much tinkering around the edges. I hereby vow to submit to CRAN by end of March at the latest! 🥅 |
Great news! |
I think I'm done with version 1.0. This update breaks a few CRAN packages that depend on janitor; I may give those maintainers some time to adjust, but otherwise I'm ready to submit what I have on GitHub now. And then maybe leave this package alone for a little while 😜 |
I submitted this AM 🤞 My previous releases have gotten quick responses, but this is a more complicated cases (it causes WARNINGs for downstream dependencies). Those cases are addressed in my |
Should this be version
1.0
? It has breaking changes, and I think represents an API paradigm shift withtabyl
. Plus honestly, I'm out of big ideas for this package after this, and don't see anything that could merit a 1.0.To dos before that:
DONE
Combine remove_empty functions, like how add_totals were combined #100,missing column names after crosstab? #77,clean_names() appends underscore before trailing number #153,1st column in a sub-result of 3-way tabyl is always a factor if show_missing_levels = TRUE #168that might be germane to this revision.increased tidyeval use #146increase tidyeval usage #178snakecase
goes to CRAN, removeRemotes:
from DESCRIPTION and addsnakecase (>= 0.9)
(confirm V number) to ImportsPENDING
AFTER SUBMITTING
90% of this release is done, now it's the dull stuff. Must. Wrap. This. Up.
The text was updated successfully, but these errors were encountered: