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.

Sign up for GitHub

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

Jump to bottom

routing/http: add more type and WithDynamicProviderInfo #443

Closed

Jorropo wants to merge 2 commits into main from http-client-types

Closed

routing/http: add more type and WithDynamicProviderInfo #443

routing/http: add more type and WithDynamicProviderInfo

Codecov / codecov/patch succeeded Aug 21, 2023 in 1s

71.42% of diff hit (target 50.11%)

View this Pull Request on Codecov

71.42% of diff hit (target 50.11%)

Annotations

Check warning on line 132 in routing/http/client/client.go

codecov / codecov/patch

routing/http/client/client.go#L131-L132

Added lines #L131 - L132 were not covered by tests

Check warning on line 174 in routing/http/client/client.go

codecov / codecov/patch

routing/http/client/client.go#L173-L174

Added lines #L173 - L174 were not covered by tests

Check warning on line 291 in routing/http/client/client.go

codecov / codecov/patch

routing/http/client/client.go#L290-L291

Added lines #L290 - L291 were not covered by tests

Check warning on line 129 in routing/http/contentrouter/contentrouter.go

codecov / codecov/patch

routing/http/contentrouter/contentrouter.go#L129

Added line #L129 was not covered by tests

View more details on Codecov