Thinkmill / manypkg

☔️ An umbrella for your monorepo
MIT License
884 stars 48 forks source link

remove six yaml-related dependencies #213

Closed benmccann closed 4 weeks ago

benmccann commented 2 months ago

Replaces https://npmgraph.js.org/?q=read-yaml-file@1.1.0 with https://npmgraph.js.org/?q=js-yaml

read-yaml-file barely does anything and is just tiny wrapper around js-yaml that calls strip-bom. However, strip-bom is unnecessary as js-yaml now strips the bom character by itself: https://github.com/nodeca/js-yaml/blob/0d3ca7a27b03a6c974790a30a89e456007d62976/lib/loader.js#L1665

changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: 324927dceb84c3e281998e2b9a22c16fb66ca8f0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | -------------- | ----- | | @manypkg/tools | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR