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

Set minimal rust version to 1.72.0 #587

Merged
merged 2 commits into from
Nov 5, 2024

Conversation

qarmin
Copy link

@qarmin qarmin commented Oct 31, 2024

This PR sets minimal rust version to 1.72.

Verification that libraries with default features compile with Rust 1.72 is included in the CI.

This version of rust is not enforced for binary projects and custom features, because they depend on external libraries that often increase the minimal versions of rust(an example is clap - 4.0(<1.72) and already 4.5(1.74))

Some tests failed on Rust 1.72.0, so I increased error size

Also due adding clippy.toml, clippy now should only report problems that don't need rust >=1.73

PR depends and contains #586

… compatibility

- Result::inspect and Option::inspect were introduced in 1.76.0
@qarmin qarmin force-pushed the set_minimial_rust_version branch 2 times, most recently from 5ee88c5 to a9837a8 Compare October 31, 2024 10:34
@Enet4 Enet4 assigned Enet4 and unassigned Enet4 Nov 1, 2024
@Enet4 Enet4 self-requested a review November 1, 2024 21:55
@Enet4 Enet4 added A-lib Area: library A-meta labels Nov 1, 2024
Copy link
Owner

@Enet4 Enet4 left a comment

Choose a reason for hiding this comment

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

Thank you for the initiative. This should take part in 0.8.0 with a minor revision.

@Enet4 Enet4 added this to the DICOM-rs 0.8 milestone Nov 1, 2024
@Enet4 Enet4 changed the title Set minimial rust version to 1.72.0 Set minimal rust version to 1.72.0 Nov 2, 2024
@qarmin qarmin force-pushed the set_minimial_rust_version branch from 681dd11 to 12a03ae Compare November 4, 2024 07:17
Copy link
Owner

@Enet4 Enet4 left a comment

Choose a reason for hiding this comment

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

Looks OK. Thank you! 👍

@Enet4 Enet4 merged commit eafc891 into Enet4:master Nov 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-lib Area: library A-meta
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants