search

amark / gun

An open source cybersecurity protocol for syncing decentralized graph data.
https://gun.eco/docs
Other
18.03k stars 1.16k forks source link

fix(npm): change glob preventing GunDataNode.d.ts #1241

Closed ShaunSHamilton closed 2 years ago

ShaunSHamilton commented 2 years ago

Changes from cd45008433759cb65476e42eabc8b076d53a9f6d caused the GunDataNode.d.ts file from being included in the NPM package.

Can be confirmed by using:

npx npm-packlist

This fix assumes the only file unwanted is stats.radata. However, this also causes the following files to be included:

test/panic/axe/data_balance_webrtc.js
test/panic/axe/data_balance.js

I left them as included, because other test files are included in the package. However, in my opinion, the npm package should not include test files 🤷‍♂️

amark commented 2 years ago

@ShaunSHamilton good catch! Thank you 👏 ⚡ ⭐ . I think long long time ago there were some data.json files but this is much safer overall. Thanks for the improvement!