Skip to content

base64ct: bump edition to 2024 (MSRV 1.85); add back to workspace #1839

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 2 commits into from
May 15, 2025

Conversation

tarcieri
Copy link
Member

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.

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 tarcieri force-pushed the base64ct/2024-edition branch from 31c5407 to d06c2f6 Compare May 15, 2025 17:50
@tarcieri tarcieri merged commit ea10f3b into master May 15, 2025
166 checks passed
@tarcieri tarcieri deleted the base64ct/2024-edition branch May 15, 2025 19:17
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