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

Use JSONL format for New Line Delimited JSON #112

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

ccoVeille
Copy link
Contributor

@ccoVeille
Copy link
Contributor Author

This is just a suggestion, it would avoid json files to be reported as in error because they contain multiple lines

@ccoVeille ccoVeille changed the title Use NDJSON format for New Line Delimited JSON Use JSONL format for New Line Delimited JSON Apr 13, 2024
@mfridman
Copy link
Owner

Nice, this should remove those pesky IDE errors related to "malformed JSON"

@mfridman
Copy link
Owner

I don't mind giving this a try, thanks for putting up a PR 🎉

@mfridman mfridman merged commit a915386 into mfridman:main Apr 16, 2024
3 checks passed
@ccoVeille ccoVeille deleted the ndjson branch April 16, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants