-
Notifications
You must be signed in to change notification settings - Fork 14
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: make clippy happy with nightly-20241008 #46
Conversation
Signed-off-by: MrCroxx <[email protected]>
Blocked by rust-lang/rust-clippy#13464 |
do you have a issue to the cippy false positive bug? |
Here is it. #46 (comment) |
ohhh. I've overseen the comment XD. you can allow the lint with a comment to the issue url. |
Signed-off-by: MrCroxx <[email protected]>
Signed-off-by: MrCroxx <[email protected]>
Signed-off-by: MrCroxx <[email protected]>
Bypass the check on the nightly toolchain with |
fastrace/examples/asynchronous.rs
Outdated
@@ -36,6 +36,8 @@ async fn other_job() { | |||
} | |||
} | |||
|
|||
// FIXME: Remove `expect` after the issue is fixed. https://github.com/rust-lang/rust-clippy/issues/13458 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please allow the lint in fastrace/Cargo.toml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
Signed-off-by: MrCroxx <[email protected]>
This PR fixed the elided lifetime check. But the needless return warning seems to be a false-positive bug.