feat!: `parseNi` etc. now return `ResolvedCommand` object instead of string, migrate to `tinyexec`

antfu-collective / ni

💡 Use the right package manager

MIT License

6.25k stars 208 forks source link

Closed userquin closed 2 weeks ago

userquin commented 4 weeks ago

This PR

parseNi, parseNr etc programmatic APIs now returns ResolvedCommand object instead of plain string
replaces @jsdevtools/ez-spawn with tinyexec using x arguments (split command using space).

🚨🚨 DO NOT RELEASE YET WE NEED TO DO SOME EXTERNAL TESTS 🚨🚨

pkg-pr-new[bot] commented 4 weeks ago

commit: 4514af5

pnpm add https://pkg.pr.new/antfu-collective/ni/@antfu/ni@231

benmccann commented 4 weeks ago

We should probably wait for https://github.com/tinylibs/tinyexec/issues/29 to be resolved first, but besides that it looks good to me

antfu commented 2 weeks ago

Made some refactoring and upgraded package-manager-detector, LGTM now