-
Notifications
You must be signed in to change notification settings - Fork 58
std/json does not support json5 #1265
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersstdWork in the Luau standard library portion of the codeWork in the Luau standard library portion of the code
Description
Activity
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersstdWork in the Luau standard library portion of the codeWork in the Luau standard library portion of the code
.luaurcis JSON5, butlute setup --with-luaurcfails whenever the file contains trailing commas or comments.{ "languageMode": "strict", }There is still no option like
--with-config-luau, either, so it is difficult to include comments in the configuration.However, this does not seem like a simple bug, since the issue occurs because
@lute/jsononly parses standard JSON. Perhaps@lute/jsonshould provide an option to parse JSONC as well.Considering that not only
.luaurc, but also Rojo’s.jsonfiles, use JSONC that is closer to JSON5, supporting only strict JSON may be limiting.of course, if the goal is to keep the library small, it would also be reasonable for it to support only standard JSON