Skip to content

Upgrade all crates to the 2024 edition and bump MSRV to 1.85 #1670

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

Merged
merged 37 commits into from
Feb 23, 2025

Conversation

baloo
Copy link
Member

@baloo baloo commented Feb 23, 2025

This removes x509-cert and its dependents from the workspace for now, until we get the chance to upgrade its dependencies.

@tarcieri
Copy link
Member

I’d say go ahead and do base*ct too

@baloo baloo force-pushed the baloo/edition-2024 branch 4 times, most recently from 005719e to 15a8a2c Compare February 23, 2025 08:47
@tarcieri
Copy link
Member

Let me get #1671 out first and then you can rebase on top of it

@tarcieri
Copy link
Member

@baloo #1671 is merged if you can rebase

@baloo baloo force-pushed the baloo/edition-2024 branch from 15a8a2c to 519a199 Compare February 23, 2025 15:10
@tarcieri tarcieri merged commit ed08fc0 into RustCrypto:master Feb 23, 2025
165 checks passed
@baloo baloo deleted the baloo/edition-2024 branch February 23, 2025 22:12
@tarcieri tarcieri mentioned this pull request Feb 24, 2025
baloo added a commit to baloo/formats that referenced this pull request Feb 26, 2025
Added
- derive additional traits on alphabets (RustCrypto#1578)

Changed
- MSRV 1.85 // Edition 2024 (RustCrypto#1670)
- reject zero-length decode requests (RustCrypto#1387)
- use `core::error::Error` (RustCrypto#1681)
@baloo baloo mentioned this pull request Feb 26, 2025
baloo added a commit that referenced this pull request Feb 26, 2025
Added
- derive additional traits on alphabets (#1578)

Changed
- MSRV 1.85 // Edition 2024 (#1670)
- reject zero-length decode requests (#1387)
- use `core::error::Error` (#1681)
@baloo baloo mentioned this pull request Feb 27, 2025
tarcieri added a commit that referenced this pull request May 15, 2025
We kept `base64ct` out of the toplevel workspace because we can't run
tests on it with its previous MSRV of 1.81 because that compiler can't
recognize the newer edition on all of the other crates in the workspace.

We originally attempted this upgrade in #1670 but reverted it in #1702
due to complaints we received in #1695.

Now that the needed MSRV of 1.85 is `N-2` of the current 1.87 stable,
we can hopefully perform this upgrade without receiving as many
complaints. Several other crates have bumped edition to 2024 already,
and edition upgrades are a time when MSRV bumps like this should be
expected.
tarcieri added a commit that referenced this pull request May 15, 2025
We kept `base64ct` out of the toplevel workspace because we can't run
tests on it with its previous MSRV of 1.81 because that compiler can't
recognize the newer edition on all of the other crates in the workspace.

We originally attempted this upgrade in #1670 but reverted it in #1702
due to complaints we received in #1695.

Now that the needed MSRV of 1.85 is `N-2` of the current 1.87 stable,
we can hopefully perform this upgrade without receiving as many
complaints. Several other crates have bumped edition to 2024 already,
and edition upgrades are a time when MSRV bumps like this should be
expected.
tarcieri added a commit that referenced this pull request May 15, 2025
)

We kept `base64ct` out of the toplevel workspace because we can't run
tests on it with its previous MSRV of 1.81 because that compiler can't
recognize the newer edition on all of the other crates in the workspace.

We originally attempted this upgrade in #1670 but reverted it in #1702
due to complaints we received in #1695.

Now that the needed MSRV of 1.85 is `N-2` of the current 1.87 stable,
we can hopefully perform this upgrade without receiving as many
complaints. Several other crates have bumped edition to 2024 already,
and edition upgrades are a time when MSRV bumps like this should be
expected.
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