search

chicoxyzzy / node-releases

Node.js releases data
MIT License
36 stars 11 forks source link

Remove data/raw from npm package #27

Closed ai closed 3 years ago

ai commented 3 years ago

Right now, node-releases footprint in node_modules is 254 KB. 80% came from data/raw.

I checked all node-releases dependents and none of big one use it (it is only browserslist who have >5 downloads per week).

Let’s remove it in major release and save 200 KB in most of node_modules in the planet (4 TB/week in Internet traffic according to size × downloads).

chicoxyzzy commented 3 years ago

I was also thinking about that, but wasn't sure if someone uses data/raw. I think that this needs fetch.js and build.js scripts to be merged. That also will require to update package.json a bit and maybe action workflow too (depends on changes).

chicoxyzzy commented 3 years ago

Oh, and README.md too. It also has spelling error: browserlist -> browserslist

ai commented 3 years ago

I fixed typo in docs.

I also remove spaces in JSON file. Anyway, we are using data in mostly single tool. I think formatting removal will bring more benefits for users in total, than a problem for 1-2 users per year who try to read files.

ai commented 3 years ago

Done. I updated Node.js, used new syntax, compress JSON and avoid file writing.