Fix multiparser trailing comma regression (#3405)

* Fix multiparser trailing comma regression

* Refactor trailing comma application
master
Lucas Azzola 2017-12-06 05:01:00 +11:00 committed by Simon Lydell
parent e09359d242
commit 829616dd6c
4 changed files with 46 additions and 4 deletions

View File

@ -26,12 +26,11 @@ function printSubtree(path, print, options) {
function parseAndPrint(text, partialNextOptions, parentOptions) {
const nextOptions = Object.assign({}, parentOptions, partialNextOptions, {
parentParser: parentOptions.parser,
trailingComma:
partialNextOptions.parser === "json"
? "none"
: partialNextOptions.trailingComma,
originalText: text
});
if (nextOptions.parser === "json") {
nextOptions.trailingComma = "none";
}
const ast = require("./parser").parse(text, nextOptions);
const astComments = ast.comments;
delete ast.comments;

View File

@ -0,0 +1,30 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP
exports[`trailing-comma.md 1`] = `
### Some heading
\`\`\`js
someFunctionCall(
foo,
bar,
foobar,
sometehingReallyLongAndHairy,
somethingElse,
breakNow,
);
\`\`\`
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
### Some heading
\`\`\`js
someFunctionCall(
foo,
bar,
foobar,
sometehingReallyLongAndHairy,
somethingElse,
breakNow,
);
\`\`\`
`;

View File

@ -0,0 +1 @@
run_spec(__dirname, ["markdown"], { trailingComma: "all" });

View File

@ -0,0 +1,12 @@
### Some heading
```js
someFunctionCall(
foo,
bar,
foobar,
sometehingReallyLongAndHairy,
somethingElse,
breakNow,
);
```