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

cmd/protoc-gen-go-grpc: default use_generic_streams_experimental to true #7387

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

arvindbr8
Copy link
Member

@arvindbr8 arvindbr8 commented Jul 3, 2024

relnotes for cmd/protoc-gen-go-grpc:

  • Generated code for services will produce streaming method definitions using one of six different generic types (e.g. BidiStreamingServer or ClientStreamingClient) instead of producing complete interfaces and implementations for the streaming methods. This can be disabled by setting use_generic_streams_experimental=false. Please file a bug if you encounter any issues with this behavior. This will become the default behavior in an upcoming release.

RELEASE NOTES: NONE

@arvindbr8 arvindbr8 added the Type: Behavior Change Behavior changes not categorized as bugs label Jul 3, 2024
@arvindbr8 arvindbr8 added this to the 1.66 Release milestone Jul 3, 2024
@arvindbr8 arvindbr8 requested a review from dfawley July 3, 2024 20:42
Copy link

codecov bot commented Jul 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.46%. Comparing base (4e9b596) to head (e67a330).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7387      +/-   ##
==========================================
+ Coverage   81.30%   81.46%   +0.16%     
==========================================
  Files         348      348              
  Lines       26744    26744              
==========================================
+ Hits        21744    21788      +44     
+ Misses       3800     3771      -29     
+ Partials     1200     1185      -15     

see 23 files with indirect coverage changes

@dfawley dfawley assigned arvindbr8 and unassigned dfawley Jul 3, 2024
@arvindbr8 arvindbr8 merged commit bb49a88 into grpc:master Jul 8, 2024
12 of 13 checks passed
printchard pushed a commit to printchard/grpc-go that referenced this pull request Jul 11, 2024
printchard pushed a commit to printchard/grpc-go that referenced this pull request Jul 30, 2024
printchard pushed a commit to printchard/grpc-go that referenced this pull request Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Behavior Change Behavior changes not categorized as bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants