Skip to content
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

chore: address clippy lints and bump ci clippy stable #5253

Merged
merged 3 commits into from
Mar 25, 2024

Conversation

jxs
Copy link
Member

@jxs jxs commented Mar 21, 2024

Description

superseeds #5248, sorry @drHuangMHT only saw yours now, can you review this one?
https://github.com/libp2p/rust-libp2p/actions/runs/8377129630/job/22938498078?pr=5253#step:6:843 is a bug, see here
Thanks!

@drHuangMHT
Copy link
Contributor

I'm not sure if the mixed_attribute_style lint works properly because include! essentially copies entire content of the source file. Though it probably doesn't matter because it's a generated file.
Also I'm not super familiar with the codebase so I can't really comment on some changes to the unused code.

@drHuangMHT
Copy link
Contributor

https://github.com/libp2p/rust-libp2p/actions/runs/8377129630/job/22938498078?pr=5253#step:6:843 is a bug, see here

Someone's going to file a backport on beta like what they did with unconditional_recurison. So It's not something they'd do very soon.

@drHuangMHT drHuangMHT mentioned this pull request Mar 22, 2024
4 tasks
swarm/src/connection.rs Show resolved Hide resolved
swarm/src/connection.rs Show resolved Hide resolved
Copy link
Contributor

@thomaseizinger thomaseizinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Nice cleanup :)

@jxs jxs added the send-it label Mar 24, 2024
@jxs
Copy link
Member Author

jxs commented Mar 24, 2024

thanks Thomas!

@mergify mergify bot merged commit 404a622 into libp2p:master Mar 25, 2024
71 of 72 checks passed
guillaumemichel pushed a commit that referenced this pull request Mar 28, 2024
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.

3 participants