Skip to content

Update to clap 4#648

Open
mkeeter wants to merge 3 commits intomasterfrom
mkeeter/update-clap
Open

Update to clap 4#648
mkeeter wants to merge 3 commits intomasterfrom
mkeeter/update-clap

Conversation

@mkeeter
Copy link
Copy Markdown
Contributor

@mkeeter mkeeter commented May 1, 2026

(Staged on #646)

This PR updates clap to version 4!

  • parse(try_from_str = parse_int::parse) becomes value_parser = parse_int::parse::<u32>
  • Conflicts and dependencies now use the original Rust names (i.e. requires = "force_dump_agent" instead of requires = "force-dump-agent")
  • I couldn't get the external subcommand to work, so I'm just accumulating cmd: Vec<String> in the tail position of the arguments
  • ArgEnum becomes ValueEnum

@mkeeter mkeeter requested a review from labbott May 1, 2026 20:24
@mkeeter mkeeter mentioned this pull request May 1, 2026
@mkeeter mkeeter force-pushed the mkeeter/update-indicatif branch from c171e69 to b70718a Compare May 1, 2026 20:30
@mkeeter mkeeter force-pushed the mkeeter/update-clap branch from 4997eb5 to 4c0aaf1 Compare May 1, 2026 20:30
Base automatically changed from mkeeter/update-indicatif to master May 1, 2026 20:51
@mkeeter mkeeter force-pushed the mkeeter/update-clap branch from 4c0aaf1 to 9035f6a Compare May 1, 2026 20:51
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.

1 participant