-
Notifications
You must be signed in to change notification settings - Fork 14
Feature/simdjson utils #58
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #58 +/- ##
==========================================
+ Coverage 94.28% 95.95% +1.66%
==========================================
Files 17 18 +1
Lines 1312 1408 +96
==========================================
+ Hits 1237 1351 +114
+ Misses 75 57 -18
Continue to review full report at Codecov.
|
…overage on some older tests, and update ChangeLog.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good as usual!
Do you want to flip it from draft to genuine PR? Or are there more parts you think are missing? |
Sure thing. I was just waiting for the CI to finish, but we should be good. |
Which seems to start in slo-mo these days. [ And because I have more than $THRESHOLD repos I can't even auto-migrate to travis-ci.com. Rock, meet hard place. ] Any reason not to fold this up and ship it to CRAN? (After one more round of win-builder / rhub of course.) |
Nope, I can't think of anything. |
Alrighty --merging and moving right along then. |
Wrapped up and shipped to CRAN. Tickled a 'needs human review' because (I think) the Windows box has now GitHub PAT and hits link-access limits 😿 as well as possibly to two existing build failures on the old box. I would expect it to fly through once they get to it, likely tomorrow (European hours) morning. |
@eddelbuettel This is largely complete.
It adds
is_valid_utf8()
,is_valid_json()
, andfminify()
(I don't think there's a built-in way to do anfprettify()
at the moment).They're all vectorized (no more
vapply(json, jsonlite::validate, logical(1L))
!) and work oncharacter
s,raw
s, andlist
s ofraw
s.I need to step away and come back with fresh eyes, but all that should be needed is a fresh coat of paint on the documentation with some examples and to ensure the arguments are sufficiently validated (it's not quite as rigorous as
fload()
/fparse()
yet).Needless to say, the wizards working upstream have made everything obscenely fast...