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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump minimum declared versions of dependencies #404

Merged
merged 2 commits into from Mar 24, 2024

Conversation

dextero
Copy link
Contributor

@dextero dextero commented Mar 21, 2024

When updating bitflags dependency of another crate, I found out that bitflags' declared dependency versions are lower than needed to actually build the crate.

This PR adds a GH workflow check inspired by the one from gbm.rs that attempts a build with the minimal compatible versions of all dependencies.

Another commit in the PR bumps the declared versions to earliest ones I identified as working, to make the workflow pass.

The workflow builds all features other than core and rustc-dep-of-std. cargo check --all --all-features --all-targets failed for me on main with mentions of rustc/std, which successfully scared me away 馃槅 I have zero experience hacking on rustc/std so I don't even know how those are used.

Hopefully this, or some parts of it can be useful as is 馃槃

Runs cargo check on nightly requesting a build with minimum versions of
dependencies declared as supported.
To minimal versions required to build most features (all other than core
and rustc-dep-of-std) with --all-targets.

Minimal required versions discovered by manual binsearch between
minimum compatible version declared currently and the newest one
available, and rerunning

  cargo check --all \
    --features serde,arbitrary,bytemuck,std,example_generated \
    --all-targets -Z minimal-versions

to verify if it builds.
Copy link
Member

@KodrAus KodrAus left a comment

Choose a reason for hiding this comment

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

Thanks @dextero! This is a great improvement. The updates to CI are much appreciated.

@KodrAus KodrAus merged commit 393be7b into bitflags:main Mar 24, 2024
10 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.

None yet

2 participants