Skip to content
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

README docs for GCS has piping instead of using upload/download? #668

Closed
jgeewax opened this issue Jun 17, 2015 · 3 comments
Closed

README docs for GCS has piping instead of using upload/download? #668

jgeewax opened this issue Jun 17, 2015 · 3 comments
Assignees
Labels
api: storage Issues related to the Cloud Storage API.

Comments

@jgeewax
Copy link
Contributor

jgeewax commented Jun 17, 2015

Seems liek this should use bucket.upload and file.download right ?

@jgeewax jgeewax added docs api: storage Issues related to the Cloud Storage API. labels Jun 17, 2015
@stephenplusplus
Copy link
Contributor

The point of showing the piping is to say "Look how Node we are!" By showing we use regular read/write streams lets developers see how they can integrate it with other libraries/architectures they may already be using.

@jgeewax
Copy link
Contributor Author

jgeewax commented Jun 17, 2015

Got it... Can we add the upload/download pieces though also? Maybe put them first, then also have the piping example?

Seems like people aren't discovering those methods like they should be.

@stephenplusplus
Copy link
Contributor

Definitely. Good idea!

@stephenplusplus stephenplusplus self-assigned this Jul 25, 2015
sofisl pushed a commit that referenced this issue Nov 11, 2022
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/c36c6dbc-ab79-4f17-b70b-523b420b2a70/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@5747555
sofisl pushed a commit that referenced this issue Nov 17, 2022
* fix!: drop support for Node.js 10

* fix!: drop support for Node.js 10

* fix!: drop support for Node.js 10

* 🦉 Updates from OwlBot

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

* 🦉 Updates from OwlBot

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

* 🦉 Updates from OwlBot

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

* 🦉 Updates from OwlBot

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

* 🦉 Updates from OwlBot

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

* chore: do not run on Node 10

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Release-As: 3.1.0
sofisl pushed a commit that referenced this issue Jan 10, 2023
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
sofisl pushed a commit that referenced this issue Jan 24, 2023
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/5fc9e824-cf7f-4f93-9bfe-6ba3aa84ec69/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@ba9918c
sofisl pushed a commit that referenced this issue Jan 25, 2023
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/5fc9e824-cf7f-4f93-9bfe-6ba3aa84ec69/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@ba9918c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the Cloud Storage API.
Projects
None yet
Development

No branches or pull requests

2 participants