Clarify goal of the spec + trailing commas seperate page#7
Conversation
|
|
Thank you, I think this is a useful clarification. I personally won't adhere to this specification then since I don't consider it useful for my requirements, but I'm glad there seems to be no room for confusion anymore. |
Alhadis
left a comment
There was a problem hiding this comment.
Editorial nits, some of them less important than others.
Co-authored-by: John Gardner <gardnerjohng@gmail.com>
|
Thanks @Alhadis for taking the time to make a review. I hope I've addressed your comments and suggestions in a satisfactionary manner. I'll probably merge this soon, but feel free to reply to my comments. Furthermore, there's also the option to discuss and make edits in another PR if pertinent. |
|
I think somehow accepting changes via the GitHub interface (DecimalTurn@2615c83) is causing the pr-preview to fail 😔, but I'm not too worried... |
The additions here aim to clarify the goal of the spec:
Provide a definition in accordance with jsonc-parser.
To avoid any ambiguity regarding whether or not a parser needs to implement trailing commas, a separate page is introduced to add more details about them, while keeping the spec short and clear. This should address https://github.com/DecimalTurn/JSONC/issues/5.