Don't commit auto-generated code

FredKSchott / rollup-plugin-polyfill-node

A modern Node.js polyfill for your Rollup bundle.

Other

175 stars 55 forks source link

Don't commit auto-generated code #20

Open letmaik opened 3 years ago

letmaik commented 3 years ago

src/polyfills.ts and polyfills/constants.js are generated during npm run build so there shouldn't be a need to commit them to the repo.

rpadaki commented 3 years ago

This also deals with the fact that constants.js has different values depending on your OS!

I'm currently trying to write a PR on Linux, and I had to manually build everything and skip the build:constants step to avoid completely rewriting constants.js and generating an unnecessary diff.

FredKSchott commented 2 years ago

I would love a PR to ignore