Skip to content

transport: change *http2Client to interface ClientTransport#7512

Merged
easwars merged 1 commit intogrpc:masterfrom
printchard:client-transport
Aug 15, 2024
Merged

transport: change *http2Client to interface ClientTransport#7512
easwars merged 1 commit intogrpc:masterfrom
printchard:client-transport

Conversation

@printchard
Copy link
Copy Markdown
Contributor

Change transport.Stream to reference the generic interface ClientTransport instead of the specific *http2Client.

RELEASE NOTES: none

@printchard printchard added the Type: Internal Cleanup Refactors, etc label Aug 15, 2024
@printchard printchard added this to the 1.66 Release milestone Aug 15, 2024
@printchard printchard requested a review from easwars August 15, 2024 00:11
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (master@9706bf8). Learn more about missing BASE report.
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #7512   +/-   ##
=========================================
  Coverage          ?   81.76%           
=========================================
  Files             ?      360           
  Lines             ?    27533           
  Branches          ?        0           
=========================================
  Hits              ?    22513           
  Misses            ?     3813           
  Partials          ?     1207           
Files Coverage Δ
internal/transport/transport.go 94.55% <ø> (ø)

@easwars easwars merged commit 5d07b63 into grpc:master Aug 15, 2024
infovivek2020 pushed a commit to infovivek2020/grpc-go that referenced this pull request Aug 18, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants