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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update clap to v4 #1489

Merged
merged 6 commits into from
Feb 22, 2023
Merged

chore: update clap to v4 #1489

merged 6 commits into from
Feb 22, 2023

Conversation

vaind
Copy link
Collaborator

@vaind vaind commented Feb 22, 2023

Following the migration guard from the changelog: https://github.com/clap-rs/clap/blob/master/CHANGELOG.md#400---2022-09-28

This was even more painful than the deprecations in v3.2

The biggest change was that all args with v4 now take a value by default, instead of being flags.

@kamilogorek
Copy link
Contributor

Tbh I tried to do this one day and gave up, as I've seen no real value in moving to v4 😅

@vaind
Copy link
Collaborator Author

vaind commented Feb 22, 2023

Tbh I tried to do this one day and gave up, as I've seen no real value in moving to v4 😅

Well I've been too fed up with the trailing spaces in the help files constantly being removed when I change something (and my editor persistently removing it) so I tried updating... not exactly as easy as I've hoped for, but it's "done", at least...

Copy link
Contributor

@kamilogorek kamilogorek left a comment

Choose a reason for hiding this comment

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

I really hope no subcommand was missed in the tests 🤞

@vaind vaind merged commit e5d8b27 into master Feb 22, 2023
@vaind vaind deleted the chore/update-clap-v4 branch February 22, 2023 15:38
vaind added a commit that referenced this pull request Mar 2, 2023
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.

2 participants