Use mdast-util-newline-to-break

[remcohaszing]

Initial checklist

• I read the support docs
• I read the contributing guide
• I agree to follow the code of conduct
• I searched issues and couldn’t find anything (or linked relevant results below)
• If applicable, I’ve added docs and tests

Description of changes

Replace the transformer with mdast-util-newline-to-break.

Closes #9

[ChristianMurphy]

Thanks @remcohaszing!
Remind me did the upstream package make any changes to the API?
I'm specifically asking to better understand the semver implications of this change.

[remcohaszing]

The upstream API should be the same, but the implementation is based on #9 (comment).

Note that it’s a drop-in replacement. No tests were modified.

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (a343c6c) 100.00% compared to head (74db9b1) 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files

@@            Coverage Diff            @@
##              main       #10   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           49        15   -34     
=========================================
- Hits            49        15   -34     

Impacted Files	Coverage Δ
index.js	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[wooorm]

Released in 3.0.3!

[absoludity]

Thanks @remcohaszing! Remind me did the upstream package make any changes to the API? I'm specifically asking to better understand the semver implications of this change.

AFAICT, the use of mdast-util-newline-to-break, which is ESM only, forces Typescript projects that import the module to update to ESM output, so not sure it should have been a patch release in retrospect?

[wooorm]

All our packages are esm for years already. So no.

Where did this comment come from? Did something break? How did you find this somewhat old issue?

[absoludity]

Sorry - I'd simply failed to include the new dep (mdast-util-newline-to-break) in our transformIgnorePatterns. Sorry for the noise.

[wooorm]

Oh is this remix still being broken 😬😬