Open Strengthless opened 1 year ago
I tried bun remove <package-name>
and saw that it executes the postinstall script.
❯ bun remove zod
bun remove v1.0.0 (822a00c4)
patch-package 8.0.0
Applying patches...
@remix-run/dev@1.19.3 ✔
@remix-run/react@1.19.3 ✔
- zod
1 packages removed [38.00ms]
This PR is now ready to merge, with only two caveats as mentioned in https://github.com/ds300/patch-package/pull/490#issue-1889202083.
@ds300 Can you please review this PR? :)
Looking forward to this :) Been using pnpm for a while and been missing patch-package - so having it work with bun will be yummy!
Hoping this will work in monorepos as well!
any news on this?
Wish I could use patch-package to patch patch-package with this patch 😂 I guess I just need to fork the repo.
Until this is merged, would you consider publishing this fork on npm @Strengthless?
LGTM!
It's happening!
Using this method for now https://github.com/oven-sh/bun/issues/2336#issuecomment-1928032858
@ds300 I understand you probably have lots going on - but having this merged would be a key enabler! 🙏🙂
Until this is merged, would you consider publishing this fork on npm @Strengthless?
@robertherber I am trying to use this method right now and published it under @miblanchard/patch-package@8.0.12 until this is merged and published.
Closes #489.
Changelogs:
--use-bun
flag, similar to the current--use-yarn
flag.parseBunLockfile.ts
to convert bun.lockb into yarn v1 lockfiles.getPackageResolution.ts
to handle bun conversions before parsing.detectPackageManager.ts
to detect bun, bun workspaces and multiple lockfiles.Caveats:
Currently awaiting confirmation on how Bun handles postinstalls, for writing up the README about bun setup. (Willbun remove
execute postinstall scripts? If not, bun users should also installpostinstall-postinstall
just like yarn v1 users.)Temporary workaround (while this PR is unmerged):