-
-
Notifications
You must be signed in to change notification settings - Fork 623
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: migrate from Node.js streams to Web Streams API #7457
Conversation
f7073c3
to
46ca103
Compare
46ca103
to
fa61643
Compare
constructor( | ||
logger?: SerialLogger, | ||
) { | ||
super(new BootloaderScreenParserTransformer(logger)); |
Check warning
Code scanning / CodeQL
Superfluous trailing arguments Warning
function TransformStream
Copilot Autofix AI 19 days ago
Copilot could not generate an autofix suggestion
Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.
logger?: SerialLogger, | ||
) { | ||
const transformer = new SerialAPIParserTransformer(logger); | ||
super(transformer); |
Check warning
Code scanning / CodeQL
Superfluous trailing arguments Warning
function TransformStream
Copilot Autofix AI 19 days ago
Copilot could not generate an autofix suggestion
Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.
constructor( | ||
logger?: SerialLogger, | ||
) { | ||
super(new ZnifferParserTransformer(logger)); |
Check warning
Code scanning / CodeQL
Superfluous trailing arguments Warning
function TransformStream
Copilot Autofix AI 19 days ago
Copilot could not generate an autofix suggestion
Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.
@zwave-js-bot automerge |
Continuing the work of making
node-zwave-js
more portable, this PR refactors the usage of streams to be compatible with the Web Streams API, rather than Node.js streams.