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

Replace ndjson with jsonl #259

Open
mpkorstanje opened this issue Oct 27, 2024 · 0 comments
Open

Replace ndjson with jsonl #259

mpkorstanje opened this issue Oct 27, 2024 · 0 comments

Comments

@mpkorstanje
Copy link
Contributor

🤔 What's the problem you're trying to solve?

The ndjson spec is defunct (ndjson/ndjson-spec#35 (comment))

✨ What's your proposed solution?

It looks like jsonl is a suitable replacement.

https://jsonlines.org/

  • Replace all mentions of ndjson
  • Update all file extensions to jsonl
  • Create issues for down stream projects to do the same.

⛏ Have you considered any alternatives or workarounds?

We can leave it as is, but that will likely cause problems.

📚 Any additional context?

No response

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

No branches or pull requests

1 participant