fix(deps): update dependency grpc-gcp to ^0.3.0

[renovate-bot]

This PR contains the following updates:

Package	Type	Update	Change
grpc-gcp	dependencies	minor	^0.1.1 -> ^0.3.0

---

Release Notes

GoogleCloudPlatform/grpc-gcp-node

v0.3.0

Compare Source

• Rework library to be compatible with grpc-js (#​67)

---

Renovate configuration

📅 Schedule: "after 9am and before 3pm" (UTC).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR has been generated by WhiteSource Renovate. View repository job log here.

[skuruppu]

'grpc-gcp' module can't be found according to the test failures.

src/index.ts:42:8 - error TS2307: Cannot find module 'grpc-gcp'.

42 } from 'grpc-gcp';
          ~~~~~~~~~~

@murgatroid99 if I could get you to take a look at the way we're doing the import, that would be really helpful. I tried to follow your comments in GoogleCloudPlatform/grpc-gcp-node#67 but I wasn't able to figure out how to get the imports working.

nodejs-spanner/src/index.ts

Line 42 in f0c6be5

} from 'grpc-gcp';

[codecov]

Codecov Report

Merging #885 into master will increase coverage by 0.36%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #885      +/-   ##
==========================================
+ Coverage   97.74%   98.10%   +0.36%     
==========================================
  Files          20       20              
  Lines       18266    17964     -302     
  Branches      965      989      +24     
==========================================
- Hits        17854    17624     -230     
+ Misses        410      339      -71     
+ Partials        2        1       -1     

Impacted Files	Coverage Δ
src/index.ts	100.00% <0.00%> (ø)
src/database.ts	99.90% <0.00%> (+0.04%)	⬆️
src/common-grpc/service.ts	100.00% <0.00%> (+0.09%)	⬆️
src/transaction-runner.ts	99.15% <0.00%> (+0.28%)	⬆️
src/codec.ts	100.00% <0.00%> (+0.32%)	⬆️
src/common-grpc/service-object.ts	100.00% <0.00%> (+0.65%)	⬆️
src/v1/spanner_client.ts	99.20% <0.00%> (+0.88%)	⬆️
src/v1/instance_admin_client.ts	99.04% <0.00%> (+0.95%)	⬆️
src/v1/database_admin_client.ts	99.33% <0.00%> (+1.02%)	⬆️

---

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 b3e0cf5...16971ee. Read the comment docs.

[murgatroid99]

The new version of grpc-gcp uses export =, so you need to import it with import = require(). What it exports is a function that takes as its argument a grpc implementation and returns the objects exported by the previous version of grpc-gcp. So, using it in typescript would look something like this:

import getGrpcGcpObjects = require('grpc-gcp');
const grpcGcp = getGrpcGcpObjects(grpc);

@alexander-fenster has told me that google-gax exports a reference to the implementation of grpc that it is using, so that should normally be the argument to this function. I expect that like the other client libraries, Spanner will want to accept a grpc implementation reference as a client construction argument. If so, that object should be passed in to both this function and google-gax.

[alexander-fenster]

Yes, gax exports the gRPC implementation. I will take care of this.

[skuruppu]

Hmmm, I attempted to fix the import in 16971ee but it didn't work :(

I'm afraid I don't know how to fix this.

[forking-renovate]

PR has been edited

👷 This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

[alexander-fenster]

This one is in my plans for the next week. We need to update google-gax dependency since it now exports grpc properly. I will take this one.

[skuruppu]

Thanks @alexander-fenster for taking this on. AFAICT, this is not a breaking change so would it be ok if I do the 5.0.0 release without this?

[forking-renovate]

Autoclosing Skipped

This PR has been flagged for autoclosing, however it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.