-
Notifications
You must be signed in to change notification settings - Fork 584
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
deprecation(streams): rework deprecations #4199
Conversation
@@ -35,7 +35,7 @@ export type { Reader, ReaderSync }; | |||
* } | |||
* ``` | |||
* | |||
* @deprecated (will be removed after 1.0.0) Use {@linkcode ReadableStream} instead. | |||
* @deprecated (will be removed in 0.215.0) Use {@linkcode ReadableStream} instead. | |||
*/ | |||
export async function* iterateReader( |
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.
Reader
is usually present alongside a ReadableStream
, and therefor ReadableStreamDefaultReader
.
@@ -20,7 +20,7 @@ import type { Reader } from "../io/types.ts"; | |||
* await copy(reader, file); | |||
* ``` | |||
* | |||
* @deprecated (will be removed after 1.0.0) Use {@linkcode ReadableStreamDefaultReader} directly. | |||
* @deprecated (will be removed after 0.215.0) Use {@linkcode ReadableStreamDefaultReader} directly. |
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.
The Streams API, when used with std/streams
, has the same functionality as std/io
. There shouldn't be a reason for someone to go from using streams to Reader
/Writer
interfaces.
|
||
/** | ||
* Options for {@linkcode writableStreamFromWriter}. | ||
* | ||
* @deprecated (will be removed after 1.0.0) Use {@linkcode WritableStream} directly. | ||
* @deprecated (will be removed in 0.215.0) Use | ||
* {@linkcode https://deno.land/std/io/to_writable_stream.ts | toWritableStream} |
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.
This is simply a correction of the deprecation notice + moving the version forward.
I'll come back to this. |
This change reworks
std/streams
deprecations for various reasons (see comments). I think it's reasonable to bring some of the removal versions forward because: