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

Fix double-version command in xtask #51

Merged
merged 1 commit into from
May 7, 2024

Conversation

alilleybrinker
Copy link
Collaborator

This commit fixes a runtime error that was stopping cargo xtask from running at all. Basically, we define a custom --version command, which was conflicting with the one automatically generated by clap, resulting in a runtime failure.

This command just adds a line telling clap not to generate its own version flag, resolving the error and allowing xtask to run as expected.

This commit fixes a runtime error that was stopping `cargo xtask` from
running at all. Basically, we define a custom `--version` command, which
was conflicting with the one automatically generated by `clap`,
resulting in a runtime failure.

This command just adds a line telling `clap` not to generate its own
`version` flag, resolving the error and allowing `xtask` to run as
expected.

Signed-off-by: Andrew Lilley Brinker <[email protected]>
@alilleybrinker alilleybrinker added the type: bug Something isn't working label May 7, 2024
@alilleybrinker alilleybrinker added this to the 3.2.0 milestone May 7, 2024
@alilleybrinker alilleybrinker requested a review from mchernicoff May 7, 2024 18:30
@alilleybrinker alilleybrinker self-assigned this May 7, 2024
@alilleybrinker alilleybrinker merged commit 52bb072 into main May 7, 2024
4 checks passed
@alilleybrinker alilleybrinker deleted the alilleybrinker/fix-xtask-run branch May 7, 2024 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants