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

Add "relaxed" json support #4

Closed
14 tasks
madeso opened this issue May 28, 2019 · 1 comment
Closed
14 tasks

Add "relaxed" json support #4

madeso opened this issue May 28, 2019 · 1 comment

Comments

@madeso
Copy link
Owner

madeso commented May 28, 2019

All of these should be optional so we can support both relaxed and strict json.

Root JSON type

Comment

Object

Array

String

Number

Based on Tencent/rapidjson#36

@madeso
Copy link
Owner Author

madeso commented Feb 12, 2021

Closing as there are already issues for all important points.

  • Storing comments aren't useful so ignoring that.
  • String without quotes already has a issue
  • Infinity/nan sounds less useful and it is probably bad practice for writing it in a config file

@madeso madeso closed this as completed Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant