-
Notifications
You must be signed in to change notification settings - Fork 264
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
tower: fix annoying clippy lints (#639)
This fixes a bunch of minor clippy lints. None of them were particularly major, but I was getting tired of the warnings showing up in vscode. The one lint that had to be ignored rather than fixed is the `clippy::bool_assert_comparison` lint, which triggers on the `tower_test::assert_request_eq!` macro. The lint triggers when writing code like `assert_eq!(whatever, true)` rather than simply `assert!(whatever)`. In this case, this occurs because the macro makes an assertion about a request value, and in _some_ tests, the request type is `bool`. We can't change this to use `assert!`, because in most cases, when the request is not `bool`, we actually do need `assert_eq!`, so I ignored that warning.
- Loading branch information
Showing
7 changed files
with
17 additions
and
9 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters