nginxinc / ngx-rust

Rust binding for NGINX
Apache License 2.0
752 stars 62 forks source link

MSRV policy #96

Open bavshin-f5 opened 1 week ago

bavshin-f5 commented 1 week ago

At the moment, we neither set the rust-version in the manifests nor test it in the CI. It would be nice to address both items. I'm still undecided between 1.82 (17 October 2024) and 1.79 (13 June 2024) and don't see compelling arguments to go below that. I'm sure these will be quite old by the time we consider the project stable.

Non-exhaustive list of benefits we could get from the recent Rust releases (as per https://releases.rs/):

1.83.0 (unreleased)

1.82.0

1.81.0

1.80.0

1.79.0

1.78.0

1.77.0

1.76.0

1.75.0

1.71.0

1.63.0

bavshin-f5 commented 11 hours ago

1.82 seems too new after all. 1.79 is old enough to be available even in RHEL, so #98 sets the MSRV to 1.79.0 and adds the CI validation.

Leaving the issue open because we want to document some kind of policy, even as arbitrary as "n-2" or "what's available in $distro".