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

Clippy lints #8233

Closed
wants to merge 15 commits into from
Closed

Conversation

brianc118
Copy link
Contributor

@brianc118 brianc118 commented Jun 26, 2024

Run clippy on all the crates.

Differential Revision: D59065418

Differential Revision: D59021487
Differential Revision: D59021488
Differential Revision: D59021491
Summary:
Format automatically gives you ToString. If we directly implement
ToString the to_string_trait_impl Clippy lint will be fired:
https://rust-lang.github.io/rust-clippy/master/index.html#/to_string_trait_impl

Differential Revision: D59021486
Differential Revision: D59021489
Differential Revision: D59021482
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 26, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59065418

@brianc118 brianc118 changed the title common: clippy lints Clippy lints Jun 26, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59065418

@brianc118 brianc118 force-pushed the export-D59065418 branch 2 times, most recently from b2cbe18 to 650b995 Compare June 26, 2024 19:16
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59065418

brianc118 and others added 9 commits June 26, 2024 13:55
Differential Revision: D59021483
Differential Revision: D59021480
Differential Revision: D59021484
Differential Revision: D59021490
Differential Revision: D59021485
Differential Revision: D59021481
Differential Revision: D59065420
Differential Revision: D59065419
Differential Revision: D59065418
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59065418

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants