-
Notifications
You must be signed in to change notification settings - Fork 9
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
Hotfixes for Hackathon #71
Conversation
Merge main into devel
Update default character_types
…valid-format Closes #69 add time formats
@elimillera Can you package up and send to CRAN this week? We should update the changelog. I'll put that on my list. |
Codecov Report
@@ Coverage Diff @@
## main #71 +/- ##
==========================================
+ Coverage 68.85% 69.35% +0.50%
==========================================
Files 10 10
Lines 366 372 +6
==========================================
+ Hits 252 258 +6
Misses 114 114
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@bms63 I'll do the final checks on devel and send it in. |
@bms63 We just need the version bumped and we are good to go |
I think the current version on CRAN is 0.1 and devel has 0.2 |
Add logic to convert any null or NA formats to "" for haven.
There is one other bug to fix. #69. Do either of you have bandwidth?
Perhaps if that is updated we could do a quick release to CRAN.