Skip to content

Commit

Permalink
Removed collections.js dependency from number-allocator.
Browse files Browse the repository at this point in the history
Since number-allocator https://www.npmjs.com/package/number-allocator
1.0.9, js-sdsl https://www.npmjs.com/package/js-sdsl is used insteaad of
collections https://www.npmjs.com/package/collections.

collections modify intrinsic type such as Array, it has unexpected side
effect for users. So I removed the dependency of collections.

It is a partial fix for mqttjs#1392.

Replacement for LRU map is still needed to remove collections dependency.
  • Loading branch information
redboltz committed Jan 5, 2022
1 parent f04c24b commit 0d95a5c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
"inherits": "^2.0.3",
"minimist": "^1.2.5",
"mqtt-packet": "^6.8.0",
"number-allocator": "^1.0.8",
"number-allocator": "^1.0.9",
"pump": "^3.0.0",
"readable-stream": "^3.6.0",
"reinterval": "^1.1.0",
Expand Down

0 comments on commit 0d95a5c

Please sign in to comment.