-
Notifications
You must be signed in to change notification settings - Fork 93
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
Pre release 0.21.0 #1426
Pre release 0.21.0 #1426
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW, we should trim out the empty sections for releases that have gone out already in the release notes. (I see some in the last couple.)
1adeba2
to
fa9cfba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
fa9cfba
to
8e1812a
Compare
Working on working around clap; need to specify a version so we dont need to bump our msrv again. |
8e1812a
to
105deab
Compare
Ok I think I have fixed clap now; Will merge once CI passes |
I don't really understand one of the commits from this PR: The commit message is "update dependencies", but the commit does not update dependencies, it restricts the versions of clap that can be used. This now broke the package because we updated clap from version 4.4 to 4.5. |
This reverts the `Cargo.toml` part of ee82223 ("cargo: update dependencies") which limited the clap version to 4.4. But we should be fine to pull in a newer version now that we bumped our MSRV. See also: coreos#1426 (comment) See also: https://src.fedoraproject.org/rpms/rust-coreos-installer/pull-request/65
This reverts the `Cargo.toml` part of ee82223 ("cargo: update dependencies") which limited the clap version to 4.4. But we should be fine to pull in a newer version now that we bumped our MSRV. See also: coreos#1426 (comment) See also: https://src.fedoraproject.org/rpms/rust-coreos-installer/pull-request/65
@decathorpe Thanks. Follow-up in #1453. |
#1425