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

Migrate to Rust 1.61.0 (+WSL fixes) #1849

Merged
merged 9 commits into from
Jun 1, 2022
Merged

Migrate to Rust 1.61.0 (+WSL fixes) #1849

merged 9 commits into from
Jun 1, 2022

Conversation

R3DP1XL
Copy link

@R3DP1XL R3DP1XL commented May 24, 2022

No description provided.

@R3DP1XL R3DP1XL force-pushed the wsl-rust-1.61-fixes branch from d0d9d14 to a57f0d3 Compare May 24, 2022 12:21
@R3DP1XL
Copy link
Author

R3DP1XL commented May 24, 2022

Still some warnings related to clippy::use_self in code generated by

  • serde (Serialize derive)
  • strum (IntoStaticStr derive)

@jelmansouri
Copy link
Contributor

I have another PR that updates dependency maybe something that was already fixed?

@jelmansouri
Copy link
Contributor

#1847

@R3DP1XL
Copy link
Author

R3DP1XL commented May 24, 2022

I have another PR that updates dependency maybe something that was already fixed?

I don't think so unfortunately, I tried updating serde&strum to latest locally, but didn't remove warnings.
Might need to make the fixes to their code generators and submit them to those crates.

@jelmansouri
Copy link
Contributor

Is it this: serde-rs/serde@2e38e2b

@R3DP1XL
Copy link
Author

R3DP1XL commented May 25, 2022

Is it this: serde-rs/serde@2e38e2b

No it's related to use_self, in code generated for enums

@jelmansouri
Copy link
Contributor

#1886

@R3DP1XL R3DP1XL force-pushed the wsl-rust-1.61-fixes branch from 17c7c06 to efd837d Compare May 31, 2022 11:36
@R3DP1XL
Copy link
Author

R3DP1XL commented Jun 1, 2022

@jelmansouri
Copy link
Contributor

rust-lang/rust-clippy#6902

@R3DP1XL do we disable this lint for now ? or we wait for 1.62 ?

@R3DP1XL
Copy link
Author

R3DP1XL commented Jun 1, 2022

rust-lang/rust-clippy#6902

@R3DP1XL do we disable this lint for now ? or we wait for 1.62 ?

I would go for a general disable, with a note that we re-enable for 1.62

@R3DP1XL R3DP1XL force-pushed the wsl-rust-1.61-fixes branch from 29e2395 to b28ffaf Compare June 1, 2022 11:01
@R3DP1XL R3DP1XL requested a review from jelmansouri June 1, 2022 12:07
@R3DP1XL R3DP1XL merged commit 672d644 into main Jun 1, 2022
@R3DP1XL R3DP1XL deleted the wsl-rust-1.61-fixes branch June 1, 2022 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants