Skip to content
This repository has been archived by the owner on Mar 26, 2024. It is now read-only.

Add an MSRV policy to the README #116

Merged
merged 1 commit into from
May 17, 2023
Merged

Add an MSRV policy to the README #116

merged 1 commit into from
May 17, 2023

Conversation

elasticdog
Copy link
Member

@elasticdog elasticdog commented May 17, 2023

This is a bit of a tricky topic, as the Rust ecosystem is waiting on better Cargo support to make usability better, and there are a lot of differing opinions and theories on how best to handle an MSRV. Not a lot of concrete data out there. The Rust compiler itself explicitly only supports the latest stable version, so any MSRV policy is essentially how long you're willing to wait for dependents of your library to upgrade before you adopt new features. Artificially holding this back might also mean missing out on security fixes, and I'm a big fan of staying current when possible.

This can always be changed, but I wanted to have something in place to make sure that it's covered.

See:

Checklist

  • Ran cargo xtask fixup for project formatting and style
  • Modified all relevant documentation
  • Updated CHANGELOG.md per "Keep a Changelog" format
  • Added tests covering my changes

This is a bit of a tricky topic, as the Rust ecosystem is waiting on
better Cargo support to make usability better, and there are a lot of
differing opinions and theories on how best to handle an MSRV. Not a
lot of concrete data out there. The Rust compiler itself explicitly only
supports the latest stable version, so any MSRV policy is essentially
how long you're willing to wait for dependents of your library to
upgrade before you adopt new features. Artificially holding this back
might also mean missing out on security fixes, and I'm a big fan of
staying current when possible.

This can always be changed, but I wanted to have something in place to
make sure that it's covered.

See:
- rust-lang/libs-team#72
- time-rs/time#535
@elasticdog elasticdog added this pull request to the merge queue May 17, 2023
Merged via the queue into main with commit 5f7e500 May 17, 2023
@elasticdog elasticdog deleted the push-xywsotqrmqks branch May 17, 2023 16:04
@elasticdog elasticdog added the documentation Improvements or additions to documentation label May 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant