Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MSRV: bump to 1.66.0 #810

Merged
merged 2 commits into from
Sep 11, 2023
Merged

MSRV: bump to 1.66.0 #810

merged 2 commits into from
Sep 11, 2023

Conversation

piodul
Copy link
Collaborator

@piodul piodul commented Sep 11, 2023

One of our indirect dependencies (toml_edit) has updated its MSRV. As we do not keep Cargo.lock committed in the repo and cargo does not honor MSRV during crate resolution, we can just bump our current MSRV by one version forward, from 1.65.0 to 1.66.0.

The version 1.66.0 is over 8 months old, so it's pretty old by the MSRV-breaking standards of the community.

As a bonus, we get rid of unzip_option workaround and use the real thing, i.e. Option::unzip().

Pre-review checklist

  • I have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • [ ] I added relevant tests for new features and bug fixes.
  • All commits compile, pass static checks and pass test.
  • PR description sums up the changes and reasons why they should be introduced.
  • [ ] I have provided docstrings for the public items that I want to introduce.
  • [ ] I have adjusted the documentation in ./docs/source/.
  • [ ] I added appropriate Fixes: annotations to PR description.

One of our indirect dependencies (`toml_edit`) has updated its MSRV. As
we do not keep Cargo.lock committed in the repo and cargo does not honor
MSRV during crate resolution, we can just bump our current MSRV by one
version forward, from 1.65.0 to 1.66.0.

The version 1.66.0 is over 8 months old, so it's pretty old by the
MSRV-breaking standards of the community.
`unzip_option` was a workaround for missing `Option::unzip()`, which was
introduced in Rust v1.66.0. We finally bumped the MSRV to 1.66.0, so
`unzip_option` can be removed and replaced with the real thing.
Copy link
Contributor

@muzarski muzarski left a comment

Choose a reason for hiding this comment

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

LGTM

@piodul piodul merged commit e9af9a1 into scylladb:main Sep 11, 2023
8 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