Description

group containing hardline or breakParent only make sense if the rest of the code relies on the Group type to make a different decision.

This one doesn't affect any test.

I also found one here:

which could be replaced by

  const linebreak =
    options.htmlWhitespaceSensitivity === "ignore"
      ? hardline
      : leadingWhitespace && trailingWhitespace
        ? line
        : null;

  if (linebreak) {
    return linebreak !== hardline
      ? group(["`", indent([linebreak, group(contentDoc)]), linebreak, "`"])
      : ["`", indent([linebreak, group(contentDoc)]), linebreak, "`"];
  }

and another here:

  return hasNewlineBetweenOrAfterDecorators(node, options)
    ? [join(line, path.map(print, "decorators")), hardline]
    : group([join(line, path.map(print, "decorators")), line]);
  ]);

But the benefit is so small that you can make that decision.

Checklist

• I’ve added tests to confirm my change works.
• (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory).
• (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/XXXX.md file following changelog_unreleased/TEMPLATE.md.
• I’ve read the contributing guidelines.

✨Try the playground for this PR✨