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

Edition 2024 #13632

Closed
wants to merge 1 commit into from
Closed

Conversation

samueltardieu
Copy link
Contributor

@samueltardieu samueltardieu commented Oct 30, 2024

This PR enables testing with Rust 2024 in the CI. It requires other PRs to be merged first, so I'll let it as a draft until this is done:

changelog: none

@rustbot
Copy link
Collaborator

rustbot commented Oct 30, 2024

r? @Jarcho

rustbot has assigned @Jarcho.
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 rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 30, 2024
@samueltardieu samueltardieu force-pushed the edition-2024 branch 2 times, most recently from 57876c0 to 2522a03 Compare October 31, 2024 12:46
@flip1995
Copy link
Member

flip1995 commented Oct 31, 2024

Not sure, if adding CI for this makes sense. The planned nightly release of the new edition is planned for in about 3-4 weeks. Then we might as well just move Clippy completely (assuming the linked PRs are merged.

But feel free to keep this PR open, if you just want to test/show on CI that everything is ready.

@samueltardieu
Copy link
Contributor Author

Yes, this was the idea, to keep it open and rebase it from time to time after those PRs are merged, so that we get a feeling of how ready we are.

@samueltardieu samueltardieu force-pushed the edition-2024 branch 3 times, most recently from 74a4c6b to 418689b Compare November 3, 2024 19:52
@Jarcho
Copy link
Contributor

Jarcho commented Nov 7, 2024

r? ghost

Since this isn't intended to be merged.

@rustbot
Copy link
Collaborator

rustbot commented Nov 7, 2024

Failed to set assignee to ghost: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@Jarcho
Copy link
Contributor

Jarcho commented Nov 7, 2024

r? @ghost

@rustbot
Copy link
Collaborator

rustbot commented Nov 7, 2024

Failed to set assignee to ghost: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@Jarcho Jarcho removed their assignment Nov 7, 2024
@bors
Copy link
Contributor

bors commented Nov 7, 2024

☔ The latest upstream changes (presumably #13587) made this pull request unmergeable. Please resolve the merge conflicts.

@samueltardieu samueltardieu force-pushed the edition-2024 branch 2 times, most recently from be30fcc to fa3e737 Compare November 11, 2024 23:48
@samueltardieu samueltardieu force-pushed the edition-2024 branch 2 times, most recently from 7ce1db6 to b523591 Compare November 23, 2024 22:33
@flip1995
Copy link
Member

Looks like we are ready for edition 2024? 👀

@samueltardieu
Copy link
Contributor Author

Looks like we are ready for edition 2024? 👀

Yes, all Clippy code in master compile without warning from the compiler or from Clippy itself. Nothing will have to be done, but selecting edition 2024 in Cargo.toml files.

@samueltardieu
Copy link
Contributor Author

Closing because of #13751

@samueltardieu samueltardieu deleted the edition-2024 branch November 29, 2024 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants