Simplify build / install tool (bun / yarn / npm) logic

rails / jsbundling-rails

Bundle and transpile JavaScript in Rails with esbuild, rollup.js, or Webpack.

MIT License

831 stars 143 forks source link

Simplify build / install tool (bun / yarn / npm) logic #180

Closed ksylvest closed 8 months ago

ksylvest commented 8 months ago

The prior logic for selecting a tool (either bun / yarn / npm) became confusing with multiple tools (and the JS ecosystem has many tools). This simplifies the logic and fixes a subtle bug where a package-lock.json is ignored if bun is installed. This issue seems to have been introduced with:

https://github.com/rails/jsbundling-rails/pull/179/files