Skip to content

Conversation

@Himess
Copy link
Contributor

@Himess Himess commented Mar 14, 2025

Description

This PR updates deprecated #[clap(...)] attributes to their modern equivalents in clap 4.x.
The current codebase still uses outdated syntax that has been deprecated since version 4.0.
By making this update, we ensure compatibility with future versions and maintain code quality.

Fixes #2002

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.
    • CHANGELOG has been updated.

….)] to #[arg(...)] and #[command(...)] in v4.x
@Himess
Copy link
Contributor Author

Himess commented Mar 14, 2025

image

There were 4 errors in the GitHub workflow, one of which was the change in Changelog.md. This was required. I have now fixed it, and we can run the workflow again.

@JulianGCalderon
Copy link
Contributor

Hi @Himess! Thanks for the contribution. Don't worry for the remaining failing jobs, they won't succeed as they only run for internal contributors.

Copy link
Contributor

@JulianGCalderon JulianGCalderon left a comment

Choose a reason for hiding this comment

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

LGTM

@gabrielbosio gabrielbosio added this pull request to the merge queue Mar 19, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Mar 19, 2025
@JulianGCalderon JulianGCalderon added this pull request to the merge queue Mar 19, 2025
Merged via the queue into lambdaclass:main with commit 3aed532 Mar 19, 2025
181 of 184 checks passed
gabrielbosio pushed a commit that referenced this pull request May 21, 2025
…#[command(...)] in v4.x (#2003)

* Update main.rs

* Update main.rs

* Update CHANGELOG.md to refactor: Clap attribute macros from #[clap(...)] to #[arg(...)] and #[command(...)] in v4.x

---------

Co-authored-by: Julián González Calderón <[email protected]>
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.

Migrate #[clap(...)] to #[arg(...)] / #[command(...)] in clap 4.x

3 participants