-
Notifications
You must be signed in to change notification settings - Fork 335
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
Cosmwasm-check: update clap to version 4 #1677
Cosmwasm-check: update clap to version 4 #1677
Conversation
a6a5019
to
1919095
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.
Thanks for the contribution!
I think it looks good, in general. Just two questions I'd like to see addressed.
311aecd
to
99bd9c2
Compare
.get_one::<String>("CAPABILITIES") | ||
.map(|s| s.as_str()) |
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.
Curious construct; but it works.
cec8123
to
657d7f6
Compare
@babadro Could you resolve the changelog conflict and add the PR link to the changelog entry? (see the other entries) |
657d7f6
to
433f570
Compare
We just wrapped up the work to get it merged. Thanks a lot for your contribution! |
Description
Closes: #1518
cosmwasm-check: