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

refactor(transport): format import and init slice cap #2741

Merged
merged 3 commits into from
Mar 26, 2023

Conversation

demoManito
Copy link
Member

Description (what this PR does / why we need it):

image

Which issue(s) this PR fixes (resolves / be part of):

Other special notes for the reviewers:

@@ -68,4 +68,4 @@ linters-settings:
# recommend 10-20
min-complexity: 50
goimports:
local-prefixes: github.com/go-kratos/kratos # Put imports beginning with prefix after 3rd-party packages
local-prefixes: github.com/go-kratos # Put imports beginning with prefix after 3rd-party packages
Copy link
Member Author

@demoManito demoManito Mar 17, 2023

Choose a reason for hiding this comment

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

github.com/go-kratos put together?

	"github.com/go-kratos/aegis/subset"
	"github.com/go-kratos/kratos"

shenqidebaozi
shenqidebaozi previously approved these changes Mar 17, 2023
@shenqidebaozi shenqidebaozi enabled auto-merge (squash) March 17, 2023 16:47
@shenqidebaozi shenqidebaozi enabled auto-merge (squash) March 17, 2023 16:48
daemon365
daemon365 previously approved these changes Mar 20, 2023
@codecov-commenter
Copy link

codecov-commenter commented Mar 21, 2023

Codecov Report

Merging #2741 (5b54513) into main (f47a238) will increase coverage by 0.02%.
The diff coverage is 87.50%.

❗ Current head 5b54513 differs from pull request most recent head 9112029. Consider uploading reports for the commit 9112029 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #2741      +/-   ##
==========================================
+ Coverage   84.78%   84.81%   +0.02%     
==========================================
  Files          88       88              
  Lines        3891     3891              
==========================================
+ Hits         3299     3300       +1     
+ Misses        418      417       -1     
  Partials      174      174              
Impacted Files Coverage Δ
transport/grpc/client.go 83.62% <ø> (ø)
transport/grpc/resolver/discovery/builder.go 90.00% <ø> (ø)
transport/grpc/server.go 83.83% <ø> (ø)
transport/http/binding/bind.go 100.00% <ø> (ø)
transport/http/codec.go 60.37% <ø> (ø)
transport/http/server.go 74.50% <ø> (ø)
transport/grpc/resolver/discovery/resolver.go 66.10% <83.33%> (+2.30%) ⬆️
transport/http/binding/encode.go 100.00% <100.00%> (ø)
transport/http/resolver.go 96.34% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@shenqidebaozi shenqidebaozi dismissed stale reviews from daemon365 and themself via 60f7b05 March 21, 2023 07:33
@demoManito demoManito changed the title fix(transport): format import and init slice cap refactor(transport): format import and init slice cap Mar 22, 2023
@shenqidebaozi shenqidebaozi merged commit d470886 into go-kratos:main Mar 26, 2023
@demoManito demoManito deleted the fix/transport/codestyle branch June 25, 2023 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants