Skip to content

refactor: Use std::io::Error::other(_) and declare MSRV 1.74#25

Merged
baszalmstra merged 2 commits intoprefix-dev:mainfrom
musicinmybrain:std-io-error-other
Feb 9, 2026
Merged

refactor: Use std::io::Error::other(_) and declare MSRV 1.74#25
baszalmstra merged 2 commits intoprefix-dev:mainfrom
musicinmybrain:std-io-error-other

Conversation

@musicinmybrain
Copy link
Contributor

Simplify a few expressions as suggested by clippy; requires Rust 1.74.

Simplify a few expressions as suggested by clippy; requires Rust 1.74.
Copy link
Collaborator

@baszalmstra baszalmstra left a comment

Choose a reason for hiding this comment

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

If this requires a particular rust version lets add it to the Cargo.toml as the minimum supported rust version!

This version is needed for `std::io::Error::other(_)`; I verified that
everything else still works on Rust 1.74 as well.
@musicinmybrain musicinmybrain changed the title refactor: Use std::io::Error::other(_) refactor: Use std::io::Error::other(_) and declare MSRV 1.74 Aug 6, 2025
@wolfv
Copy link
Member

wolfv commented Feb 9, 2026

@baszalmstra should we merge this?

@baszalmstra baszalmstra merged commit 914f690 into prefix-dev:main Feb 9, 2026
5 checks passed
@baszalmstra
Copy link
Collaborator

Yes, thanks for the ping!

@github-actions github-actions bot mentioned this pull request Feb 9, 2026
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.

3 participants