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

bump dependency versions to the minimum version that compiles. #428

Merged
merged 2 commits into from
Dec 21, 2024

Conversation

ericseppanen
Copy link
Contributor

@ericseppanen ericseppanen commented Dec 21, 2024

The dependency versions in Cargo.toml were inaccurate in places, because they allowed versions that don't actually work-- crates were enabling Cargo features that don't exist in those versions, or methods/traits/etc that didn't exist in the code.

This allows a tool like cargo minimal-versions to verify that the code compiles even when using the oldest possible dependency versions, but is also a useful step toward verifying that the crates can be built on older toolchains.

Replaces #419.

These dependency versions are the minimum that can actually build. This
allows a tool like `cargo minimal-versions` to verify that the code
compiles even when using the oldest possible dependency versions, but is
also a useful step toward verifying that the crates can be built on
older toolchains.
@Eugeny
Copy link
Owner

Eugeny commented Dec 21, 2024

Looking good! I've added a CI job to run minimal-versions as well

@Eugeny
Copy link
Owner

Eugeny commented Dec 21, 2024

@all-contributors add @ericseppanen for code

@Eugeny Eugeny merged commit 039054b into Eugeny:main Dec 21, 2024
7 checks passed
Copy link
Contributor

@Eugeny

I've put up a pull request to add @ericseppanen! 🎉

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