Closed jankapunkt closed 1 year ago
@TheRealNate with this release we will have 0 vulnerabilities, a pretty good coverage and even a few bugs fixed due to #111
Coverage report:
-------------------------|---------|----------|---------|---------|----------------------
File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
-------------------------|---------|----------|---------|---------|----------------------
All files | 85.67 | 80.69 | 81.54 | 86.33 |
helpers | 63.63 | 100 | 33.33 | 75 |
reactNativeBindings.js | 63.63 | 100 | 33.33 | 75 | 14-16
lib | 93.01 | 88.77 | 96.87 | 93.15 |
Random.js | 100 | 100 | 100 | 100 |
ddp.js | 86.15 | 87.5 | 89.47 | 87.5 | 43-59
mongo-id.js | 100 | 100 | 100 | 100 |
queue.js | 100 | 100 | 100 | 100 |
sha256.js | 92.59 | 40 | 100 | 92 | 206-212
socket.js | 100 | 100 | 100 | 100 |
utils.js | 84.61 | 88 | 100 | 84.61 | 36,69,75-76
src | 81.71 | 77.51 | 73.26 | 82.31 |
Call.js | 100 | 75 | 100 | 100 | 5
Collection.js | 76.77 | 72 | 73.68 | 77.85 | ...3,262,291,302-317
Data.js | 23.68 | 35.71 | 6.25 | 23.68 | 15-81,92-93
ReactiveDict.js | 100 | 100 | 100 | 100 |
Tracker.js | 92 | 81.63 | 94.44 | 93.15 | ...4,411,512,534,620
-------------------------|---------|----------|---------|---------|----------------------
Seems like it will be a big help! Approved, LGTM.
It's merged but I still can't publish to NPM registry, can you do that or add me to the ord on NPM (I have 2FA active on NPM)
@jankapunkt sure can do, same username?
@TheRealNate thanks I just got the invitation and will publish now
Hm seems to be revoked, anyway my account is https://www.npmjs.com/~jkuester
Added!
This merges the current dev state into
master
, making 2.4.0 the new main HEAD.