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

[0.2] Explicitly set the edition to 2015 #4058

Merged
merged 1 commit into from
Nov 16, 2024

Conversation

tgross35
Copy link
Contributor

This just suppresses a warning. Ideally this should go to 2021 but that requires quite a bit of refactoring, so hold off until the rest of the 0.2 cleanup is complete.

This just suppresses a warning. Ideally this should go to 2021 but that
requires quite a bit of refactoring, so hold off until the rest of the
0.2 cleanup is complete.
@rustbot
Copy link
Collaborator

rustbot commented Nov 16, 2024

r? @JohnTitor

rustbot has assigned @JohnTitor.
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

@tgross35 tgross35 changed the title Explicitly set the edition to 2015 [0.2] Explicitly set the edition to 2015 Nov 16, 2024
@tgross35 tgross35 added this pull request to the merge queue Nov 16, 2024
Merged via the queue into rust-lang:libc-0.2 with commit be62010 Nov 16, 2024
44 checks passed
tgross35 added a commit to tgross35/rust-libc that referenced this pull request Nov 17, 2024
This just suppresses a warning. Ideally this should go to 2021 but that
requires quite a bit of refactoring, so hold off until the rest of the
0.2 cleanup is complete.

(apply <rust-lang#4058> to `main`)
(cherry picked from commit 1e88f41)
@tgross35 tgross35 added the main-applied This PR was merged against libc-0.2 and main has since been updated to include it label Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
main-applied This PR was merged against libc-0.2 and main has since been updated to include it S-waiting-on-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants