New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: use built-in exec
instead of execa
with execGit
#1379
base: master
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: a36161e The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
Ping @okonet do you have any opinion on reducing the number of dependencies? This Draft MR is a trial on whether current fresh Node.js LTS versions would allow us to drop Hoping that Windows tests pass.... 🤞 |
I'm fine with it. Looks nice indeed. Why are user tasks need still to be wrapped with execa? Can they use this wrapper? Or did I misunderstand this part? |
This was easier to test with, thus the draft PR. For user tasks it will still need:
|
That makes sense. I'm wondering if it then worth the change if we keep the dependency in place. Can these missing bits be done just with Node? |
Should be possible. I'll keep this as a draft until it's done. |
No description provided.