-
Notifications
You must be signed in to change notification settings - Fork 27
Conversation
Not seeing how its blocking this PR |
Npm build fails 😅 |
Is this blocked by #278 ? |
To make sure there's no chance we end up copying memory to a new location only to pass it to a stream handler, return `Uint8ArrayList`s for the `data` property of multiplexed messages. The stream interface looks like this: ``` Stream extends Duplex<Uint8ArrayList, Uint8ArrayList | Uint8Array> ``` That is, it's a Duplex whose source emits `Uint8ArrayList`s but the sink can accept `Uint8ArrayList`s or `Uint8Arrays`. Depends on: - [ ] libp2p/js-libp2p-interfaces#279
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
## [@libp2p/interface-metrics-v3.0.0](https://github.com/libp2p/js-libp2p-interfaces/compare/@libp2p/interface-metrics-v2.0.0...@libp2p/interface-metrics-v3.0.0) (2022-08-07) ### ⚠ BREAKING CHANGES * change stream muxer interface (#279) ### Features * change stream muxer interface ([#279](#279)) ([1ebe269](1ebe269)) ### Trivial Changes * update project config ([#271](#271)) ([59c0bf5](59c0bf5))
🎉 This PR is included in version @libp2p/interface-metrics-v3.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
## [@libp2p/interface-connection-v3.0.0](https://github.com/libp2p/js-libp2p-interfaces/compare/@libp2p/interface-connection-v2.1.1...@libp2p/interface-connection-v3.0.0) (2022-08-07) ### ⚠ BREAKING CHANGES * change stream muxer interface (#279) * change connection encryption interface to uint8arraylist (#278) ### Features * change connection encryption interface to uint8arraylist ([#278](#278)) ([1fa580c](1fa580c)) * change stream muxer interface ([#279](#279)) ([1ebe269](1ebe269)) ### Trivial Changes * update project config ([#271](#271)) ([59c0bf5](59c0bf5))
🎉 This PR is included in version @libp2p/interface-connection-v3.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
## [@libp2p/interface-stream-muxer-compliance-tests-v4.0.0](https://github.com/libp2p/js-libp2p-interfaces/compare/@libp2p/interface-stream-muxer-compliance-tests-v3.0.1...@libp2p/interface-stream-muxer-compliance-tests-v4.0.0) (2022-08-07) ### ⚠ BREAKING CHANGES * change stream muxer interface (#279) ### Features * change stream muxer interface ([#279](#279)) ([1ebe269](1ebe269)) ### Trivial Changes * update project config ([#271](#271)) ([59c0bf5](59c0bf5)) ### Dependencies * update sibling dependencies ([f859920](f859920))
🎉 This PR is included in version @libp2p/interface-stream-muxer-compliance-tests-v4.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
## [@libp2p/interface-mocks-v4.0.0](https://github.com/libp2p/js-libp2p-interfaces/compare/@libp2p/interface-mocks-v3.0.3...@libp2p/interface-mocks-v4.0.0) (2022-08-07) ### ⚠ BREAKING CHANGES * change stream muxer interface (#279) * change connection encryption interface to uint8arraylist (#278) ### Features * change connection encryption interface to uint8arraylist ([#278](#278)) ([1fa580c](1fa580c)) * change stream muxer interface ([#279](#279)) ([1ebe269](1ebe269)) ### Dependencies * update sibling dependencies ([f75e927](f75e927)) * update sibling dependencies ([d98a5ea](d98a5ea)) * update sibling dependencies ([f859920](f859920)) * update sibling dependencies ([93a89b1](93a89b1))
🎉 This PR is included in version @libp2p/interface-mocks-v4.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@wemeetagain @achingbrain multistream-select is blocking updating muxer interface to use uint8arraylist