Skip to content

Conversation

@AVaksman
Copy link
Contributor

@AVaksman AVaksman commented Mar 2, 2020

  • tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Towards googleapis/nodejs-storage/issues/956 as a prerequisite to googleapis/nodejs-storage/pull/1074
to provide the same functionality as gcs-resumable-upload

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 2, 2020
Copy link

@bcoe bcoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks reasonable to me, minus nit.

test/util.ts Outdated
import {replaceProjectIdToken} from '@google-cloud/projectify';
import * as assert from 'assert';
import {describe, it} from 'mocha';
// import {describe, it} from 'mocha';
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we get rid of this comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@bcoe
Copy link

bcoe commented Mar 9, 2020

@AVaksman land whenever you're ready 👍

@AVaksman AVaksman merged commit 1834059 into googleapis:master Mar 9, 2020
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