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

chore: Merge multitransport client and fixes #836

Merged
merged 9 commits into from
Sep 7, 2021

Conversation

vam-google
Copy link
Contributor

No description provided.

arithmetic1728 and others added 9 commits August 17, 2021 11:56
* feat: enable self signed jwt for gapic clients

* resolve comments

* update gax version

* update goldens

* update golden files
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
This includes:
1) Proper GRPC transcoding
2) Mimimal REST Operations (go make it compilable)
3) Rely on resource names for tests values generation (to not fail http url path validation)
4) Empty Server Streaming method implementation
1) Fully backward compatible with grpc-only GAPIC
2) Strictly speaking not fully backward compatible with rest-only REGAPIC (but is compatible for practical cases). See below for details.
3) Introduces the concept of default transport (is necessary to preserve backward-compabitility). The default behavior assumes grpc transport.

Needed to unblock DIREGAPIC LRO. To be reviewed after submission
@vam-google vam-google requested review from miraleung and a team as code owners September 7, 2021 07:28
@codecov
Copy link

codecov bot commented Sep 7, 2021

Codecov Report

❗ No coverage uploaded for pull request base (diregapic-lro@957f69a). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 92f7c1c differs from pull request most recent head 79ef502. Consider uploading reports for the commit 79ef502 to get more accurate results
Impacted file tree graph

@@               Coverage Diff                @@
##             diregapic-lro     #836   +/-   ##
================================================
  Coverage                 ?   89.75%           
================================================
  Files                    ?      151           
  Lines                    ?    15432           
  Branches                 ?     1114           
================================================
  Hits                     ?    13851           
  Misses                   ?     1266           
  Partials                 ?      315           

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 957f69a...79ef502. Read the comment docs.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 7, 2021
@vam-google vam-google merged commit 200fce7 into googleapis:diregapic-lro Sep 7, 2021
suztomo pushed a commit that referenced this pull request Mar 21, 2023
…836)

Source-Link: googleapis/synthtool@505ce5a
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:2567a120ce90fadb6201999b87d649d9f67459de28815ad239bce9ebfaa18a74
suztomo pushed a commit that referenced this pull request Mar 21, 2023
🤖 I have created a release *beep* *boop*
---


## [2.8.0](googleapis/java-core@v2.7.1...v2.8.0) (2022-06-21)


### Features

* add build scripts for native image testing in Java 17 ([#1440](https://github.com/googleapis/java-core/issues/1440)) ([#836](googleapis/java-core#836)) ([49e9b68](googleapis/java-core@49e9b68))


### Dependencies

* update dependency com.google.api-client:google-api-client-bom to v1.35.1 ([#843](googleapis/java-core#843)) ([d81fbf0](googleapis/java-core@d81fbf0))
* update dependency com.google.api:api-common to v2.2.1 ([#844](googleapis/java-core#844)) ([d711da9](googleapis/java-core@d711da9))
* update dependency com.google.api:gax-bom to v2.18.2 ([#849](googleapis/java-core#849)) ([d44a6d9](googleapis/java-core@d44a6d9))
* update dependency com.google.api.grpc:proto-google-common-protos to v2.9.0 ([#846](googleapis/java-core#846)) ([981b634](googleapis/java-core@981b634))
* update dependency com.google.api.grpc:proto-google-iam-v1 to v1.4.0 ([#838](googleapis/java-core#838)) ([8b04fb4](googleapis/java-core@8b04fb4))
* update dependency com.google.api.grpc:proto-google-iam-v1 to v1.4.1 ([#848](googleapis/java-core#848)) ([78dd4ed](googleapis/java-core@78dd4ed))
* update dependency com.google.errorprone:error_prone_annotations to v2.14.0 ([#839](googleapis/java-core#839)) ([a4b5158](googleapis/java-core@a4b5158))
* update dependency com.google.http-client:google-http-client-bom to v1.42.0 ([#845](googleapis/java-core#845)) ([24211fb](googleapis/java-core@24211fb))
* update dependency com.google.protobuf:protobuf-bom to v3.21.0 ([#840](googleapis/java-core#840)) ([93d825f](googleapis/java-core@93d825f))
* update dependency com.google.protobuf:protobuf-bom to v3.21.1 ([#841](googleapis/java-core#841)) ([7065ef0](googleapis/java-core@7065ef0))
* update dependency io.grpc:grpc-bom to v1.47.0 ([#842](googleapis/java-core#842)) ([3245109](googleapis/java-core@3245109))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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.

None yet

2 participants