fix: replace deprecated #[clap(...)] with #[command(...)] and #[arg(...)]
Description
Erros :
cargo check --features clap/deprecated
This PR updates all deprecated #[clap(...)] and #[arg(name = ...)] usages throughout the codebase to align with clap v4 attribute style.
Changes include:
- Replaced
#[clap(...)]with#[command(...)]and#[arg(...)] - Replaced deprecated
name =withvalue_name =where appropriate
Test plan
Solve :
cargo check --features clap/deprecated
(There are still a few remaining warnings. I will create another PR to address them.)
Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.
- [ ] Protocol:
- [ ] Nodes (Validators and Full nodes):
- [ ] gRPC:
- [ ] JSON-RPC:
- [ ] GraphQL:
- [ ] CLI:
- [ ] Rust SDK:
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| sui-docs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | May 19, 2025 8:37pm |
2 Skipped Deployments
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| multisig-toolkit | ⬜️ Ignored (Inspect) | Visit Preview | May 19, 2025 8:37pm | |
| sui-kiosk | ⬜️ Ignored (Inspect) | Visit Preview | May 19, 2025 8:37pm |
@amnn Thanks for review . I will update or i can create other PR to solve it.
Best to update this PR -- we don't want to land this one with that change (it will mess up doc generation).
Got it, agreed. I’ll update it as soon as possible and tag you.
@amnn Done. Could you check again ?
@Himess it looks like there are some cases where arg was used instead of command etc. Can you take a look at the CI output and fix the issues? I will then reschedule the workflows.
@amnn When i crate PR , i didn 't get any errors. I think that happend for conflicts or smthmg. I will check again and i will update thanks !
@amnn Ready to review sir.
@amnn I think it must be work right now :) sory for delay. Could you run CI again ?
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.