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

Test that project works as cargo extension #23

Closed
eopb opened this issue Jul 29, 2024 · 2 comments · Fixed by #26
Closed

Test that project works as cargo extension #23

eopb opened this issue Jul 29, 2024 · 2 comments · Fixed by #26

Comments

@eopb
Copy link
Owner

eopb commented Jul 29, 2024

Make sure that after installing cargo-override it can be run as a cargo extension, as cargo override

@eopb
Copy link
Owner Author

eopb commented Aug 7, 2024

This is the bug I expected to see

cargo override --help
error: unexpected argument 'override' found

Usage: cargo-override --path <PATH>

For more information, try '--help'.

We need to accept an initial override arg

@eopb
Copy link
Owner Author

eopb commented Aug 8, 2024

Running cargo override results in a call to cargo-override override so we need to support this subcommand

eopb added a commit that referenced this issue Aug 8, 2024
Fixes #23

This makes it possible to run `cargo override --path ../dep` where previously `cargo-override --path ../dep` had to be used.
@eopb eopb closed this as completed in #26 Aug 8, 2024
@eopb eopb closed this as completed in b3695ec Aug 8, 2024
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 a pull request may close this issue.

1 participant