prettier 1.2.4 → 1.5.1

data/src/ruby/printer.js

@@ -38,7 +38,8 @@ const noComments = [
   "args_add_star",
   "mlhs",
   "mlhs_add_post",
-  "mlhs_add_star"
+  "mlhs_add_star",
+  "mlhs_paren"
 ];
 
 // Certain nodes are used more for organizational purposed than for actually
@@ -54,8 +55,6 @@ function getCommentChildNodes(node) {
   switch (node.type) {
     case "heredoc":
       return [node.beging];
-    case "rescue":
-      return [].concat(node.body[0]).concat(node.body.slice(1));
     case "aryptn":
       return [node.body[0]]
         .concat(node.body[1])
@@ -99,8 +98,12 @@ function getCommentChildNodes(node) {
 
       return parts;
     }
-    default:
-      return node.body;
+    default: {
+      if (Array.isArray(node.body)) {
+        return node.body.filter((child) => child && typeof child === "object");
+      }
+      return [];
+    }
   }
 }
 

data/src/utils.js

@@ -1,6 +1,7 @@
 module.exports = {
   containsAssignment: require("./utils/containsAssignment"),
   getTrailingComma: require("./utils/getTrailingComma"),
+  isEmptyBodyStmt: require("./utils/isEmptyBodyStmt"),
   isEmptyStmts: require("./utils/isEmptyStmts"),
   hasAncestor: require("./utils/hasAncestor"),
   literal: require("./utils/literal"),

data/src/utils/inlineEnsureParens.js

@@ -1,4 +1,11 @@
-const needsParens = ["args", "assign", "assoc_new", "massign", "opassign"];
+const needsParens = [
+  "args",
+  "assign",
+  "assoc_new",
+  "call",
+  "massign",
+  "opassign"
+];
 
 // If you have a modifier statement (for instance an inline if statement or an
 // inline while loop) there are times when you need to wrap the entire statement

data/src/utils/isEmptyBodyStmt.js

@@ -0,0 +1,7 @@
+const isEmptyStmts = require("./isEmptyStmts");
+
+function isEmptyBodyStmt(node) {
+  return isEmptyStmts(node.body[0]) && !node.body.slice(1).some(Boolean);
+}
+
+module.exports = isEmptyBodyStmt;

data/src/utils/isEmptyStmts.js

@@ -1,7 +1,11 @@
-const isEmptyStmts = (node) =>
-  node &&
-  node.type === "stmts" &&
-  node.body.length === 1 &&
-  node.body[0].type === "void_stmt";
+function isEmptyStmts(node) {
+  return (
+    node &&
+    node.type === "stmts" &&
+    node.body.length === 1 &&
+    node.body[0].type === "void_stmt" &&
+    !node.body[0].comments
+  );
+}
 
 module.exports = isEmptyStmts;

data/src/utils/noIndent.js

@@ -4,6 +4,7 @@ const noIndent = [
   "heredoc",
   "if",
   "method_add_block",
+  "unless",
   "xstring_literal"
 ];
 

data/src/utils/printEmptyCollection.js

@@ -1,5 +1,11 @@
 const { concat, group, hardline, indent, join, line } = require("../prettier");
 
+function containedWithin(node) {
+  return function containedWithinNode(comment) {
+    return comment.sc >= node.sc && comment.ec <= node.ec;
+  };
+}
+
 // Empty collections are array or hash literals that do not contain any
 // contents. They can, however, have comments inside the body. You can solve
 // this by having a child node inside the array that gets the comments attached
@@ -7,11 +13,12 @@ const { concat, group, hardline, indent, join, line } = require("../prettier");
 // print out the non-leading comments here.
 function printEmptyCollection(path, opts, startToken, endToken) {
   const node = path.getValue();
+  const containedWithinNode = containedWithin(node);
 
   // If there are no comments or only leading comments, then we can just print
   // out the start and end token and be done, as there are no comments inside
   // the body of this node.
-  if (!node.comments || !node.comments.some((comment) => !comment.leading)) {
+  if (!node.comments || !node.comments.some(containedWithinNode)) {
     return `${startToken}${endToken}`;
   }
 
@@ -21,7 +28,7 @@ function printEmptyCollection(path, opts, startToken, endToken) {
   const printComment = (commentPath) => {
     const comment = commentPath.getValue();
 
-    if (!comment.leading) {
+    if (containedWithinNode(comment)) {
       comment.printed = true;
       comments.push(opts.printer.printComment(commentPath));
     }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prettier
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.5.1
 platform: ruby
 authors:
 - Kevin Deisz
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-01-03 00:00:00.000000000 Z
+date: 2021-01-28 00:00:00.000000000 Z
 dependencies: []
 description:
 email:
@@ -31,8 +31,30 @@ files:
 - node_modules/prettier/third-party.js
 - package.json
 - rubocop.yml
+- src/haml/embed.js
+- src/haml/nodes/comment.js
+- src/haml/nodes/doctype.js
+- src/haml/nodes/filter.js
+- src/haml/nodes/hamlComment.js
+- src/haml/nodes/plain.js
+- src/haml/nodes/root.js
+- src/haml/nodes/script.js
+- src/haml/nodes/silentScript.js
+- src/haml/nodes/tag.js
+- src/haml/parser.js
+- src/haml/parser.rb
+- src/haml/printer.js
+- src/parser/getLang.js
+- src/parser/getNetcat.js
+- src/parser/netcat.js
+- src/parser/parseSync.js
+- src/parser/requestParse.js
+- src/parser/server.rb
 - src/plugin.js
 - src/prettier.js
+- src/rbs/parser.js
+- src/rbs/parser.rb
+- src/rbs/printer.js
 - src/ruby/embed.js
 - src/ruby/nodes.js
 - src/ruby/nodes/alias.js
@@ -75,6 +97,7 @@ files:
 - src/utils/getTrailingComma.js
 - src/utils/hasAncestor.js
 - src/utils/inlineEnsureParens.js
+- src/utils/isEmptyBodyStmt.js
 - src/utils/isEmptyStmts.js
 - src/utils/literal.js
 - src/utils/literalLineNoBreak.js