rollup / plugins

🍣 The one-stop shop for official Rollup plugins
MIT License
3.62k stars 580 forks source link

chore(repo): use shell-emulator for better cross-platform compatibility #1743

Open danielbayley opened 3 months ago

danielbayley commented 3 months ago

Rollup Plugin Name: *

This PR contains:

Are tests included?

Breaking Changes?

If yes, then include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.

List any relevant issue numbers:

Description

I started work on a separate PR, but quickly ran into errors with the prepare scripts apparently not being cross-platform (if command not found—I am on macOS). Since this is a pnpm monorepo, and the scripts are very simple, why not make use of the shell-emulator .npmrc config option (which uses yarnpkg-shell)? This also means the dependency on del-cli is no longer necessary.

tada5hi commented 3 months ago

failing wasm test should be unrelated to pr?! @shellscape

danielbayley commented 5 days ago

@shellscape Should be resolved now…