Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(deps): update dependency @google-cloud/common to v3 and handle stream ending #704

Merged
merged 4 commits into from
Apr 15, 2020

Conversation

crwilcox
Copy link
Contributor

@crwilcox crwilcox commented Apr 15, 2020

co-authored by: @bcoe

#678 fails system tests due to a change in pumpify/readable streams. Changes to which tick things are cleaned up on seems to have surfaced an unpiping issue.

co-authored by: bcoe <bencoe@google.com>
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 15, 2020
@crwilcox crwilcox changed the title fix: handle unpiping at userStream end fix(deps)!: update dependency @google-cloud/common to v3 and handle stream ending Apr 15, 2020
@crwilcox crwilcox requested a review from JustinBeckwith April 15, 2020 03:28
@crwilcox crwilcox changed the title fix(deps)!: update dependency @google-cloud/common to v3 and handle stream ending fix(deps): update dependency @google-cloud/common to v3 and handle stream ending Apr 15, 2020
@codecov
Copy link

codecov bot commented Apr 15, 2020

Codecov Report

Merging #704 into master will decrease coverage by 0.02%.
The diff coverage is 80.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #704      +/-   ##
==========================================
- Coverage   99.01%   98.99%   -0.03%     
==========================================
  Files          15       15              
  Lines       15286    15289       +3     
  Branches      935      932       -3     
==========================================
- Hits        15136    15135       -1     
- Misses        147      151       +4     
  Partials        3        3              
Impacted Files Coverage Δ
src/table.ts 99.62% <80.00%> (-0.22%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a95ab4a...cfeb6e3. Read the comment docs.

@crwilcox crwilcox merged commit d8ada04 into master Apr 15, 2020
@stephenplusplus stephenplusplus deleted the test-error-code branch April 15, 2020 17:06
@crwilcox
Copy link
Contributor Author

#708 made to make sure we remember we may want to simplify some of this.

@bcoe bcoe added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Apr 15, 2020
# for free to join this conversation on GitHub. Already have an account? # 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