pretty-quick icon indicating copy to clipboard operation
pretty-quick copied to clipboard

chore: replace `execa` with `tinyexec`

Open pralkarz opened this issue 1 year ago • 2 comments

This PR replaces execa with a lighter alternative.

Some notes:

  1. tinyexec only provides an asynchronous API. My changes aim to keep the existing behavior through async/await, while also following the ESLint rules (e.g. not inlining (await func(...)).stdout but extracting the await output to a separate variable).
  2. I went ahead and added some miscellaneous things to .gitignore for smoother DX. 😄
  3. I tackled the comments regarding mocking execaSync (now x). Please let me know if you'd prefer to keep the old approach.

pralkarz avatar Sep 04 '24 22:09 pralkarz

🦋 Changeset detected

Latest commit: 00d1b94a3bbb99a39b323a79db246cc728da6abe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
pretty-quick Minor

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

changeset-bot[bot] avatar Sep 04 '24 22:09 changeset-bot[bot]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

codesandbox-ci[bot] avatar Sep 04 '24 22:09 codesandbox-ci[bot]

Sorry I didn't notice this PR for a long time, seems great.

JounQin avatar Mar 04 '25 14:03 JounQin