Skip to content

fix: remove unpipe package and use native unpipe method#543

Merged
UlisesGascon merged 1 commit intoexpressjs:masterfrom
Phillip9587:remove-unpipe
Oct 22, 2024
Merged

fix: remove unpipe package and use native unpipe method#543
UlisesGascon merged 1 commit intoexpressjs:masterfrom
Phillip9587:remove-unpipe

Conversation

@Phillip9587
Copy link
Copy Markdown
Member

The unpipe package was used to unpipes all destinations from a given stream.

Taken from the unpipe readme:

With stream 2+, this is equivalent to stream.unpipe(). When used with streams 1 style streams (typically Node.js 0.8 and below), this module attempts to undo the actions done in stream.pipe(dest).

As our minimum supported Node version is v18 the unpipe package is no longer needed.

@UlisesGascon UlisesGascon merged commit 012330e into expressjs:master Oct 22, 2024
@UlisesGascon UlisesGascon self-assigned this Oct 22, 2024
@UlisesGascon UlisesGascon mentioned this pull request Oct 29, 2024
@Phillip9587 Phillip9587 deleted the remove-unpipe branch February 19, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants