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

2024 rust edition #1698

Merged
merged 7 commits into from
Mar 3, 2025
Merged

2024 rust edition #1698

merged 7 commits into from
Mar 3, 2025

Conversation

CommanderStorm
Copy link
Collaborator

This PR upgrades (cargo fix --eddition, then fix what does not make sense) our crates to the 2024 edition.

While quite early, this enables things such as the v3 resolver, which might be helpfull to prevent MRSV breakages in the future by a MSRV breakage now.

Most of the changes are from rustfmt => going commit-by-commit is likely a good approach

@CommanderStorm
Copy link
Collaborator Author

CommanderStorm commented Feb 22, 2025

CI working is depedent on the 2024 eddition getting to githubs CI..:

Copy link
Member

@nyurik nyurik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

once CI passes, this is good to go

@CommanderStorm CommanderStorm enabled auto-merge (squash) March 3, 2025 15:05
@CommanderStorm CommanderStorm merged commit 1deeda8 into maplibre:main Mar 3, 2025
8 of 16 checks passed
@CommanderStorm
Copy link
Collaborator Author

Why did this get automerged?
This does not pass CI?

@CommanderStorm
Copy link
Collaborator Author

In particular, we are missing this PR:

CommanderStorm added a commit that referenced this pull request Mar 3, 2025
CommanderStorm added a commit that referenced this pull request Mar 3, 2025
Reverts #1698

I have no idea why this was automerged.. that was a mistake
@CommanderStorm CommanderStorm deleted the 2024_rust branch March 4, 2025 03:02
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