7rulnik / source-map-js

Consume and generate source maps.
Other
92 stars 13 forks source link

Reduce npm package size #7

Closed ai closed 2 years ago

ai commented 2 years ago

This change reduces source-map-js footprint in node_modules from 700 KB (!) to 100 KB.

Now, source-map-js is 77% of installing size of PostCSS 😓.

Also https://github.com/7rulnik/source-map-js/pull/6 can be merged to fix now useless package.json key,

7rulnik commented 2 years ago

@ai sorry for slow response

I think we should keep webpack, because currently, it is necessary for benchmarking in a browsers. Also, we can remove API documentation and replace it with link to original source-map documentation.

And I like the idea of removing comments from published code. If you think it's safe enough, so let's do it.

7rulnik commented 2 years ago

And as I understand, it should be major release? I'm fine with it. We can release it as 1.0.0 BTW. I thought that it will be useful to try keep in sync versions with original source-map, but seems that it doesn't make any sense.

ai commented 2 years ago

I think we should keep webpack, because currently, it is necessary for benchmarking in a browsers.

Can we build webpack version just to use in benchmark and do not put file to npm package?

ai commented 2 years ago

And as I understand, it should be major release

Yeap, let’s do major (1.0 or 0.7).

ai commented 2 years ago

😍

20211105193924