Skip to content

Bump MSRV to 1.85.0 to fix CI - #130

Merged
eldruin merged 2 commits into
masterfrom
bump-msrv-fix-ci
Aug 21, 2026
Merged

Bump MSRV to 1.85.0 to fix CI#130
eldruin merged 2 commits into
masterfrom
bump-msrv-fix-ci

Conversation

@jannic

@jannic jannic commented Aug 18, 2026

Copy link
Copy Markdown
Member

One indirect dependency, unescaper, set edition="2024" in a minor release (0.1.9) without declaring a MSRV. Therefore, even with resolver.incompatible-rust-versions = "fallback", cargo will update the dependency and the CI build will fail because our MSRV doesn't support edition 2024.

Instead of locking unescaper at version 0.1.8, I prefer to bump our MSRV from 1.84.0 to 1.85.0: The next release will be a MSRV bump from 1.65.0 anyway, and it doesn't matter much if we jump to 1.84.0 or 1.85.0.

Also contains a trivial fix for a new clippy warning.

One indirect dependency, unescaper, set edition="2024" in a minor release (0.1.9)
without declaring a MSRV. Therefore, even with
`resolver.incompatible-rust-versions = "fallback"`, cargo will update
the dependency and the CI build will fail because our MSRV doesn't
support edition 2024.

Instead of locking unescaper at version 0.1.8, I prefer to bump our MSRV
from 1.84.0 to 1.85.0: The next release will be a MSRV bump from 1.65.0
anyway, and it doesn't matter much if we jump to 1.84.0 or 1.85.0.
@jannic
jannic requested a review from a team as a code owner August 18, 2026 18:33
@jannic jannic mentioned this pull request Aug 18, 2026

@eldruin eldruin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sounds good to me, thank you!

@eldruin
eldruin added this pull request to the merge queue Aug 21, 2026
Merged via the queue into master with commit 932bfdd Aug 21, 2026
22 of 23 checks passed
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