Skip to content

Cargo.toml: add MSRV of 1.87#47

Closed
jstarks wants to merge 1 commit intomicrosoft:mainfrom
jstarks:msrv
Closed

Cargo.toml: add MSRV of 1.87#47
jstarks wants to merge 1 commit intomicrosoft:mainfrom
jstarks:msrv

Conversation

@jstarks
Copy link
Member

@jstarks jstarks commented May 19, 2025

edit requires at least Rust 1.87, to support the unstable MaybeUninit::write_filled method. Update Cargo.toml to document this, so that there's a clear error message when attempting to build with an older toolchain.

(I think eliminating the use of this method (and the maybe_uninit_fill unstable feeature) would allow the MSRV to be reduced to 1.86.)

`edit` requires at least Rust 1.87, to support the unstable
`MaybeUninit::write_filled` method. Update `Cargo.toml` to document
this, so that there's a clear error message when attempting to build
with an older toolchain.

(I think eliminating the use of this method (and the `maybe_uninit_fill`
unstable feeature) would allow the MSRV to be reduced to 1.86.)
@jstarks
Copy link
Member Author

jstarks commented May 19, 2025

I see #43 already does this and more.

@lhecker
Copy link
Member

lhecker commented May 19, 2025

Yeah, I think we should close this in favor of #43, in particular as it adds the license field. What do you think?

@jstarks
Copy link
Member Author

jstarks commented May 19, 2025

Sure thing.

@jstarks jstarks closed this May 19, 2025
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