Move comment into code

fisker · fisker · commit ed020b28cacf · 2023-12-19T15:55:06.000+08:00
diff --git a/docs/options.md b/docs/options.md
@@ -115,10 +115,6 @@ Note that Prettier never unquotes numeric property names in Angular expressions,
 [quote-props-flow]: https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBjOA7AzgFzAA8wBeMAb1TDAAYAuMARlQF8g
 [quote-props-vue]: https://github.com/prettier/prettier/issues/10127
 
-<!-- TODO[@fisker]: Check this -->
-
-If this option is set to `preserve`, `singleQuote` to `false` (default value), and `parser` to `json5`, double quotes are always used for strings. This effectively allows using the `json5` parser for “JSON with comments and trailing commas”.
-
 ## JSX Quotes
 
 Use single quotes instead of double quotes in JSX.
diff --git a/src/utils/print-string.js b/src/utils/print-string.js
@@ -12,6 +12,12 @@ function printString(raw, options) {
   const enclosingQuote =
     options.parser === "json" ||
     options.parser === "jsonc" ||
+    // This was added before we have the `jsonc` parser
+    // If `{quoteProps: "preserve"}` and `{singleQuote: false}` (default value),
+    // and `parser` to `json5`, double quotes are always used for strings.
+    // This effectively allows using the `json5` parser for “JSON with comments and trailing commas”.
+    // See https://github.com/prettier/prettier/pull/10323
+    // See https://github.com/prettier/prettier/pull/15831#discussion_r1431010636
     (options.parser === "json5" &&
       options.quoteProps === "preserve" &&
       !options.singleQuote)
