Skip to content

Docs: WritableStream -> WriteStream#1401

Merged
stephenplusplus merged 3 commits intogoogleapis:masterfrom
kevinresol:patch-3
Jul 1, 2016
Merged

Docs: WritableStream -> WriteStream#1401
stephenplusplus merged 3 commits intogoogleapis:masterfrom
kevinresol:patch-3

Conversation

@kevinresol
Copy link
Copy Markdown
Contributor

I am not pretty sure about this one though.
Because it writes WriteStream in another place

Actually, in fs it is called fs.WriteStream but the underlying class/interface is stream.Writable. I am not sure what it is referring to...

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 30, 2016
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 30, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling da96c27 on kevinresol:patch-3 into b94126b on GoogleCloudPlatform:master.

@stephenplusplus
Copy link
Copy Markdown
Contributor

I think we should favor WritableStream, which is just to say we are returning a stream that is writable. Do you mind basically reversing this PR and changing it in the other file that you found WriteStream?

@kevinresol
Copy link
Copy Markdown
Contributor Author

No problem. Do you want to change ReadStream to ReadableStream as well then?

@kevinresol
Copy link
Copy Markdown
Contributor Author

I went ahead changing readstream to readable stream. Also added the missing return type for File.create(read/write)stream

@coveralls
Copy link
Copy Markdown

coveralls commented Jul 1, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 3fe8bac on kevinresol:patch-3 into b94126b on GoogleCloudPlatform:master.

@stephenplusplus
Copy link
Copy Markdown
Contributor

Perfect! Thanks again :)

@stephenplusplus stephenplusplus merged commit 61b27cb into googleapis:master Jul 1, 2016
sofisl pushed a commit that referenced this pull request Feb 25, 2026
* fix: remove disrespectful terms

* updated signatures

* updated policies and signatures

* revert v4 conformance test data changes

Co-authored-by: Frank Natividad <[email protected]>
GautamSharda pushed a commit that referenced this pull request Mar 5, 2026
* chore(main): release 10.4.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
thiyaguk09 pushed a commit to thiyaguk09/google-cloud-node-fork that referenced this pull request Mar 18, 2026
* fix: remove disrespectful terms

* updated signatures

* updated policies and signatures

* revert v4 conformance test data changes

Co-authored-by: Frank Natividad <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants