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

Update MSRV to 1.65.0 #156

Merged
merged 1 commit into from
Feb 23, 2023
Merged

Update MSRV to 1.65.0 #156

merged 1 commit into from
Feb 23, 2023

Conversation

joshlf
Copy link
Member

@joshlf joshlf commented Feb 16, 2023

Also update the versions of the stable and nightly compiler toolchains that are pinned in CI.

@joshlf joshlf force-pushed the msrv branch 6 times, most recently from 5f95320 to 6aef126 Compare February 22, 2023 23:54
@joshlf
Copy link
Member Author

joshlf commented Feb 23, 2023

Looks like the tests are failing due to rust-lang/miri#2791.

Also update the versions of the stable and nightly compiler toolchains
that are pinned in CI.
@joshlf joshlf merged commit f86bdd1 into main Feb 23, 2023
@joshlf joshlf deleted the msrv branch February 23, 2023 00:48
joshlf added a commit that referenced this pull request Aug 8, 2023
It turned out that there were only a few lines of code that required
our MSRV to be as high as it was before this commit (1.65.0), and those
lines were easy to modify to be compatible with this new MSRV.

Note that this requires re-introducing a bug that was removed in #156
because the bug is based on an underlying bug in the standard library.
That bug was considered minor at the time it was discovered, and I still
consider it minor now. See #64 for more details.
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.

1 participant