holmgr / cargo-sweep

A cargo subcommand for cleaning up unused build files generated by Cargo

MIT License

729 stars 31 forks source link

Found argument '--recursive' which wasn't expected, or isn't valid in this context #99

Closed Dueen closed 11 months ago

Dueen commented 1 year ago

The recursive argument seems to not work when specifying the long-form argument.

Machine: M1 Macbook Pro, macOS 13.2.0 Ventura

Expected behavior:

Running the command

cargo sweep --recursive --time 30 path/to/project

does the same as

cargo sweep -r --time 30 path/to/project

Actual behavior:

Running this command

cargo sweep --recursive --time 30 path/to/project

Errors out, while running the short-form of the recursive command

cargo sweep -r --time 30 path/to/project

succeeds.

marcospb19 commented 1 year ago

@Dueen what's the output of cargo sweep --version? This is working fine for me at the latest on master branch (0.6.2).

EDIT: from the error message, it seems like you're running an older version. EDIT2: latest version on crates.io is 0.6.2, but from a year ago, maybe we need a new release.

Dueen commented 1 year ago

@marcospb19 Yep, I am on v0.6.2 from crates.io

Expurple commented 12 months ago

@jyn514, is there a reason why there still haven't been any new releases on crates.io? I've just discovered this awesome tool, encountered the same issue and resorted to cargo install cargo-sweep --git https://github.com/holmgr/cargo-sweep.git as a workaround. However,

Not every potential user may come up with this on the spot.
README still recommends plain cargo install cargo-sweep, which installs a 2.5 year old version.
The crates.io page gives a wrong impression that the project is unmaintained.

marcospb19 commented 12 months ago

The crates.io page gives a wrong impression that the project is unmaintained.

Indeed, this project had some maintenance hiccups, I'm now the new main maintainer :wave::smiley:.

Thank you for the bump! I'll prioritize making a new release this week!

Expurple commented 12 months ago

@marcospb19, thanks. Can you also add a section like "Maintainers" to the README then? I wasn't quite sure when I was tagging Jynn

marcospb19 commented 12 months ago

To some people, that could be seen as if I'm self-promoting myself, so I'd like not to :v .

Additionally, that won't be a problem because I get notifications for all interactions and will be able to answer quickly.

I only mentioned I'm new to explain why I didn't make a release already (but I will soon!! :pray: )

marcospb19 commented 11 months ago

Thanks for the patience guys, I merged 6 PRs today and made the new release, it's already available on crates.io :wink:, cheers!