search

pulsar-edit / ppm

Pulsar Package Manager
MIT License
35 stars 13 forks source link

meta: Actually sync yarn.lock #60

Closed DeeDeeG closed 1 year ago

DeeDeeG commented 1 year ago

This should make yarn.lock stable across multiple installs.

(Not sure why this diff wasn't part of the npm bump, but who knows?)

DeeDeeG commented 1 year ago

Note, this technically overlaps with #57, but I think this is worth merging on its own.

This change is also committed in #57, but it is arguably a bit off-topic/out-of-scope there (?) and there is no diff between this branch and that one in terms of `yarn.lock`, so there should be no merge conflict if this one merges first. #57 is currently a WIP PR with multiple changes needing to be done, and/or its scope reduced significantly to be mergeable, and so I don't think it'll be ready right away, whereas this is a really narrow one-file diff with a small diff at that.

If possible, I'd like to get the present PR merged here before syncing back to the core Pulsar repo again.

DeeDeeG commented 1 year ago

Thank you for the review, merging this one, PR at core to bump the ppm submodule soon!