Skip to content

Comments

remove generic parameters from Reconnect::new#755

Merged
tobz merged 3 commits intotower-rs:masterfrom
GlenDC:patch/issue-737-remove-generic-params
Jul 20, 2024
Merged

remove generic parameters from Reconnect::new#755
tobz merged 3 commits intotower-rs:masterfrom
GlenDC:patch/issue-737-remove-generic-params

Conversation

@GlenDC
Copy link
Contributor

@GlenDC GlenDC commented Dec 5, 2023

these were not used, as the only parameters used
come from the impl block (directly and indirectly)

Closes #737

these were not used, as the only parameters used
come from the impl block (directly and indirectly)
@tobz tobz added A-reconnect Area: The tower "reconnect" middleware C-cleanup Category: PRs that clean code up or issues documenting cleanup. labels Jul 20, 2024
@tobz tobz merged commit 7d723eb into tower-rs:master Jul 20, 2024
@tobz tobz added relnotes Marks issues that should be documented in the release notes of the next release. S-awaiting-release Status: approved/merged but awaiting a release. labels Jul 20, 2024
@tobz
Copy link
Member

tobz commented Aug 13, 2024

This has been released as part of [email protected].

Thanks again for your contribution!

@tobz tobz removed the S-awaiting-release Status: approved/merged but awaiting a release. label Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-reconnect Area: The tower "reconnect" middleware C-cleanup Category: PRs that clean code up or issues documenting cleanup. relnotes Marks issues that should be documented in the release notes of the next release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reconnect::new() Generic parameters are redundant

2 participants