-
-
Notifications
You must be signed in to change notification settings - Fork 52
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
ENH: Support newline delimited JSON (ndjons/jsonl) #14
Comments
A JSON lines file was the first file I tried to used |
Thanks! But it seems to be a duplicate of #9, so I will close it 🙏 |
Hard to find the original as I've never heard of Json stream being used to talk about Jsonl/ndjson. Could you edit the title of the original to be clearer? Also maybe close as "duplicate" not as "completed", when I saw the purple completion I was already happy that it was added only to be disappointed that it was closed as duplicate 😇 |
Sure. But let me confirm what both issues want to say. These are adding the functionality that allows filtering over multiple JSON structures contained in a single file/stdin, right?
I apologize for changing the issue state on my own. I'll proceed in a way that everyone can agree on. Therefore, please let me reopen it (Even if it is deemed a duplicate again and gets closed) 🙇 |
@corneliusroemer In #28, I've made it possible to handle multiple JSONs. Please check whether the problem addressed in this issue has been resolved. |
@corneliusroemer Released v0.2.0 which addressed this issue. Please check it out 🎉 |
Installed this last week and didn't get to test it until today when I had a use-case I really needed it for. Encountered the trailing issue, found this issue, updated, and now it works! @ynqa thanks for a great tool :) |
I'd love to use
jnv
with newline delimited json files as well. Right now, when I load an ndjson file, I get the following error:jq
on the other hand has absolutely no issue withjsonl
files.The text was updated successfully, but these errors were encountered: