search

folke / esbuild-runner

⚡️ Super-fast on-the-fly transpilation of modern JS, TypeScript and JSX using esbuild
https://www.npmjs.com/package/esbuild-runner
Apache License 2.0
712 stars 24 forks source link

esbuild.config.js is too generic naming #22

Closed IlyaSemenov closed 3 years ago

IlyaSemenov commented 3 years ago

I believe esbuild.config.js is too generic naming. It misleads the reader as if that were esbuild config, which it is not. It also 'squats' the place for the actual esbuild config, for example if developer wanted to keep and reuse esbuild settings between different components of the project (such as: esbuild-runner, estrella, vite, esbuild-loader, etc.)

I believe esbuild-runner.config.js would be better naming, directly explaining the purpose of the file.

folke commented 3 years ago

You are totally right about this. Didn't think that one through. I'll change it!

folke commented 3 years ago

Just released a new version with the updates https://github.com/folke/esbuild-runner/releases/tag/2.0.0