Skip to content

chore: apply some clippy lints #4415

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 25, 2025
Merged

chore: apply some clippy lints #4415

merged 1 commit into from
Apr 25, 2025

Conversation

nyurik
Copy link
Contributor

@nyurik nyurik commented Apr 15, 2025

@rustbot label +stable-nominated

@rustbot
Copy link
Collaborator

rustbot commented Apr 15, 2025

r? @tgross35

rustbot has assigned @tgross35.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot
Copy link
Collaborator

rustbot commented Apr 15, 2025

Some changes occurred in OpenBSD module

cc @semarie

Some changes occurred in the Android module

cc @maurer

Some changes occurred in solarish module

cc @jclulow, @pfmooney

@rustbot rustbot added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Apr 15, 2025
@nyurik nyurik changed the title Lints chore: apply some clippy lints Apr 15, 2025
@nyurik nyurik force-pushed the lints branch 3 times, most recently from d07ad69 to 1285ebb Compare April 15, 2025 16:58
@rustbot rustbot added the O-bsd label Apr 15, 2025
@nyurik nyurik force-pushed the lints branch 2 times, most recently from b8d6959 to 38bc29a Compare April 15, 2025 18:45
@rustbot rustbot added the ctest Issues relating to the ctest crate label Apr 15, 2025
tgross35
tgross35 requested changes Apr 16, 2025
View reviewed changes
@nyurik nyurik requested a review from tgross35 April 21, 2025 14:36
tgross35
tgross35 approved these changes Apr 25, 2025
View reviewed changes
Copy link
Contributor

@tgross35 tgross35 left a comment

Choose a reason for hiding this comment

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

Looks great, thank you for working on these! As with the others, needs a squash

@tgross35 tgross35 enabled auto-merge April 25, 2025 06:59
@tgross35
Copy link
Contributor

Just to get merged I did it here :)

@tgross35 tgross35 added this pull request to the merge queue Apr 25, 2025
Merged via the queue into rust-lang:main with commit 172b20a Apr 25, 2025
52 checks passed
@nyurik nyurik deleted the lints branch April 25, 2025 15:21
@nyurik
Copy link
Contributor Author

nyurik commented Apr 25, 2025

thx :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tgross35 tgross35 tgross35 approved these changes

Assignees

@tgross35 tgross35

Labels
ctest Issues relating to the ctest crate O-android O-bsd O-dragonfly O-gnu O-linux O-linux-like O-macos O-riscv O-solarish O-unix S-waiting-on-review stable-nominated This PR should be considered for cherry-pick to libc's stable release branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nyurik @rustbot @tgross35