-
Notifications
You must be signed in to change notification settings - Fork 786
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
Update clap, and potentially move to derive #5087
Comments
I'd like to try to get this across the finish line. I'm wondering if we could do the clap upgrades in stages, i.e. separate EDIT: |
I think that's a good idea and from what I remember should be possible, haven't checked out your PR yet but will try to get to it soon (off for the next couple weeks though) |
Description
I started on moving to clap derive a while ago, so there is work started that could be picked up, but it's pretty difficult to keep up to date. We'd get a little more compile time safety, but it is a lot of changes, so not totally sure it's worth it. Relevant PRs:
#3007
#2960
The text was updated successfully, but these errors were encountered: