Skip to content

Remove bound from into_make_service_with_connect_info - #892

Merged
davidpdrsn merged 2 commits into
mainfrom
info-make-service-bound
Mar 31, 2022
Merged

Remove bound from into_make_service_with_connect_info#892
davidpdrsn merged 2 commits into
mainfrom
info-make-service-bound

Conversation

@davidpdrsn

Copy link
Copy Markdown
Member

Fixes #859

Turns out removing the bound doesn't actually hurt the compile error. So I think we can just remove it.

@davidpdrsn
davidpdrsn requested a review from jplatte March 31, 2022 16:37
@davidpdrsn
davidpdrsn enabled auto-merge (squash) March 31, 2022 16:37
@davidpdrsn
davidpdrsn merged commit 21552fe into main Mar 31, 2022
@davidpdrsn
davidpdrsn deleted the info-make-service-bound branch March 31, 2022 16:49
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.

Router::into_make_service_with_connect_info() has unnecessary Target parameter that can fail to be inferred

2 participants