Skip to content

Conversation

@starsep
Copy link
Contributor

@starsep starsep commented Nov 12, 2025

Based on astral-sh/ruff#21337
Edit: Updated to usage of clap_cargo

Before After (style from ruff) After (clap_cargo)
before ruff clap_cargo

@epage
Copy link
Collaborator

epage commented Nov 12, 2025

It looks like ruff is mimicking cargo. Would be good to copy them completely via https://docs.rs/clap-cargo/latest/clap_cargo/style/constant.CLAP_STYLING.html

@epage
Copy link
Collaborator

epage commented Nov 12, 2025

Could you clean up the commits for how they should be reviewed and merged? Intermediate states related to review feedback don't matter to me.

@starsep
Copy link
Contributor Author

starsep commented Nov 12, 2025

Sure, I hope it's alright now 🙂

@epage
Copy link
Collaborator

epage commented Nov 12, 2025

Thanks!

@epage epage enabled auto-merge November 12, 2025 22:09
@coveralls
Copy link

coveralls commented Nov 12, 2025

Pull Request Test Coverage Report for Build 19313350261

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 19.183%

Totals Coverage Status
Change from base Build 19044188569: 0.0%
Covered Lines: 535
Relevant Lines: 2789

💛 - Coveralls

@epage epage merged commit 55474f5 into crate-ci:master Nov 12, 2025
23 checks passed
@starsep starsep deleted the colorful_help branch November 12, 2025 22:37
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.

3 participants