Skip to content
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

update to simdjson 0.5.0 #49

Merged
merged 5 commits into from Oct 7, 2020
Merged

update to simdjson 0.5.0 #49

merged 5 commits into from Oct 7, 2020

Conversation

@eddelbuettel
Copy link
Owner

@eddelbuettel eddelbuettel commented Oct 6, 2020

Better late than never :)

@knapply lemme know what you think. This isn't urgent as I have been, ahem, sitting on this for a few weeks but (coughs twice) beginning of term, this, that, the other and general "it's friggin 2020" madness appear to have gotten in the way :)

eddelbuettel added 4 commits Oct 5, 2020
@eddelbuettel eddelbuettel requested a review from knapply Oct 6, 2020
@knapply
knapply approved these changes Oct 7, 2020
Copy link
Collaborator

@knapply knapply left a comment

Looks awesome to me. 🔥

I'm not seeing any commented-out tests. Are they all running again? If so, 🔥🚀

@eddelbuettel
Copy link
Owner Author

@eddelbuettel eddelbuettel commented Oct 7, 2020

I'm not seeing any commented-out tests.

Yes, I followed up on this today with a PR that just turned them from expect_error() to expect_identical() with the results they now give -- see 5330193 for details if curious.

@eddelbuettel eddelbuettel merged commit bef0257 into master Oct 7, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.