- From: Dan Sanders via GitHub <sysbot+gh@w3.org>
- Date: Wed, 12 Aug 2020 20:43:24 +0000
- To: public-webrtc-logs@w3.org
I was the source of the feedback mentioned above, and my concern was that it could be very hard to reason about the correct way to use the streams. `pipeThough()` appearing to work but not doing anything useful seems like it will cause a lot of frustration for first-time users. I think that error being present in the issue description above supports my concern. Adding a 'Stream' suffix isn't inherently more clear though, it just prevents that specific mistake and encourages users to look at the documentation. I don't know if you are considering more substantial changes, like for example inverting the relationship to be a method that takes a Transformer and sets up the pipes for you. -- GitHub Notification of comment by sandersdan Please view or discuss this issue at https://github.com/w3c/webrtc-insertable-streams/issues/40#issuecomment-673101071 using your GitHub account -- Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Wednesday, 12 August 2020 20:43:25 UTC