rollup-plugin-polyfill-node icon indicating copy to clipboard operation
rollup-plugin-polyfill-node copied to clipboard

Don't commit auto-generated code

Open letmaik opened this issue 4 years ago • 2 comments

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.

letmaik avatar May 06 '21 20:05 letmaik

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.

rpadaki avatar May 28 '21 18:05 rpadaki

I would love a PR to ignore

FredKSchott avatar Dec 03 '21 05:12 FredKSchott