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.60 and check on the CI #397

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

sdroege
Copy link
Contributor

@sdroege sdroege commented Dec 8, 2023

The code does not actually compile anymore with 1.51.

Also only run cargo check with 1.60 as various optional features, tests, benchmarks actually require newer Rust versions.

Copy link
Member

@daniel-abramov daniel-abramov left a comment

Choose a reason for hiding this comment

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

Thanks!

@sdroege
Copy link
Contributor Author

sdroege commented Dec 8, 2023

Doesn't look like cargo hack even works with that version? :)

@sdroege
Copy link
Contributor Author

sdroege commented Dec 8, 2023

Also 1.60 only works with default features and not any of the tests. So I'll limit the CI to that I guess.

@sdroege
Copy link
Contributor Author

sdroege commented Dec 8, 2023

Only running cargo check now with 1.60 as other (optional) things actually require newer Rust versions than that.

The code does not actually compile anymore with 1.51.

Also only run `cargo check` with 1.60 as various optional features,
tests, benchmarks actually require newer Rust versions.
@sdroege
Copy link
Contributor Author

sdroege commented Dec 8, 2023

@daniel-abramov All passing it seems.

@daniel-abramov daniel-abramov merged commit e9604ac into snapview:master Dec 8, 2023
6 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