refactor!: replace execa with tinyexec
Resolves #150.
This PR is a suggestion to replace heavier execa with a lighter tinyexec to reduce the dependency count and the bundle size.
Actually, since there are only two usages, we could just use node:child_process directly. What do you think?
Thanks for PR. I think so we might directly use node:child_process.
Nypm was doing this before. We moved to execa to support cross-platform search path of corepack mainly (https://github.com/unjs/nypm/pull/54). However with Node.js not going to ship corepack by default.. i think we might change strategy altogether.
Since this change might introduce regression, i am thinking more about options we have.
Thanks for PR. I think so we might directly use
node:child_process.Nypm was doing this before. We moved to execa to support cross-platform search path of
corepackmainly (#54). However with Node.js not going to ship corepack by default.. i think we might change strategy altogether.Since this change might introduce regression, i am thinking more about options we have.
No rush! No matter which solution you decide on, I'm happy to help driving this replacement forward.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 63.40%. Comparing base (
660392f) to head (f14a467). Report is 56 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #151 +/- ##
===========================================
- Coverage 82.17% 63.40% -18.77%
===========================================
Files 6 5 -1
Lines 516 470 -46
Branches 71 84 +13
===========================================
- Hits 424 298 -126
- Misses 91 169 +78
- Partials 1 3 +2
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.