-
Notifications
You must be signed in to change notification settings - Fork 203
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
chore: upgrade rustc version #1489
Conversation
Cargo.toml
Outdated
@@ -101,7 +101,7 @@ catalog_impls = { path = "src/catalog_impls" } | |||
horaedbproto = { git = "https://github.com/apache/incubator-horaedb-proto.git", rev = "19ece8f771fc0b3e8e734072cc3d8040de6c74cb" } | |||
codec = { path = "src/components/codec" } | |||
chrono = "0.4" | |||
clap = "3.0" | |||
clap = "=4.4.18" |
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.
Is it necessary to lock the version to the "4.4.18"?
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.
The latest version of clap is currently 4.5.1. However, during local testing, I found that the latest version may not be compatible with the rustc version, so the version is temporarily locked at 4.4.18.
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.
Clap is very aggressive for rust version In my impression, I wonder which features do you depend on but missing from current version?
I think our user case is simple enough that there is hardly any change we need to bump it.
ef2562e
to
0e692af
Compare
@@ -16,5 +16,5 @@ | |||
# under the License. | |||
|
|||
[toolchain] | |||
channel = "nightly-2023-08-28" |
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.
Try search nightly-2023-08-28
in our codebase, I remember some CI files also hard code it.
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.
OK
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
## Rationale Upgrade rustc version ## Detailed Changes - upgrade rustc version - upgrade rand version - upgrade obkv-client version - upgrade clap version - Adapt to new APIs ## Test Plan CI
Upgrade rustc version - upgrade rustc version - upgrade rand version - upgrade obkv-client version - upgrade clap version - Adapt to new APIs CI
Upgrade rustc version - upgrade rustc version - upgrade rand version - upgrade obkv-client version - upgrade clap version - Adapt to new APIs CI
Rationale
Upgrade rustc version
Detailed Changes
Test Plan
CI