10BC0 stream: include strategy options on Readable by Warkanlock · Pull Request #43515 · nodejs/node · GitHub

stream: include strategy options on Readable#43515

Merged

nodejs-github-bot merged 10 commits intonodejs:mainfrom

Warkanlock:strategy_for_webstream

Jul 13, 2022

Commits on Jun 21, 2022

stream: initial approach to include strategy options on Readable.toWeb()
Warkanlock
committed
stream: extend strategy option using an object to allow extensibility
Warkanlock
committed
stream: fix typo on comment and change name of parameter in the documentation file
Warkanlock
committed
stream: add unit test for strategy option
Warkanlock
committed
stream: added parameter size to calculate size of chunks if needed
Warkanlock
committed
stream: add better unit test to check if desireSize is being assigned
Warkanlock
committed
stream: remove unused imports
Warkanlock
committed
stream: remove nested nernary and replace it with an inner function
Warkanlock
committed
stream: use an arrow function instead
Warkanlock
committed

Commits on Jun 23, 2022

stream: refactor evaluateStrategyOrFallback method inspired by nit suggestions
Warkanlock
committed

0