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

Allow Tail { "Key": "Value" } Comma [@ READ ONLY] #1734

Closed
ClaudiuHKS opened this issue Sep 2, 2019 · 1 comment
Closed

Allow Tail { "Key": "Value" } Comma [@ READ ONLY] #1734

ClaudiuHKS opened this issue Sep 2, 2019 · 1 comment
Labels
kind: enhancement/improvement solution: wontfix the issue will not be fixed (either it is impossible or deemed out of scope)

Comments

@ClaudiuHKS
Copy link

Example of current use ->

Only { "Name": "Michael", "Age": 36 }

Example of use after this feature request ->

Either { "Name": "Michael", "Age": 36 } or { "Name": "Michael", "Age": 36, }

Note! This feature request should only affect the functions that read the trees. Those functions that write should remain the same.

@ClaudiuHKS ClaudiuHKS changed the title Tail { "Key" : "Value" } Comma [@ READ ONLY] Tail { "Key": "Value" } Comma [@ READ ONLY] Sep 2, 2019
@ClaudiuHKS ClaudiuHKS changed the title Tail { "Key": "Value" } Comma [@ READ ONLY] Allow Tail { "Key": "Value" } Comma [@ READ ONLY] Sep 2, 2019
@nlohmann
Copy link
Owner

nlohmann commented Sep 2, 2019

This library only accepts valid JSON, and trailing commas are not allowed, see

@nlohmann nlohmann closed this as completed Sep 2, 2019
@nlohmann nlohmann added the solution: wontfix the issue will not be fixed (either it is impossible or deemed out of scope) label Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: enhancement/improvement solution: wontfix the issue will not be fixed (either it is impossible or deemed out of scope)
Projects
None yet
Development

No branches or pull requests

2 participants