Closed mattico closed 1 year ago
As for --time
's help text I agree that it's strange since an argument is required and I didn't notice any logic regarding some fallback value either (the current approach .expect()
s a time value along with .expect()
ing it being a valid u64
As for the version flag being weird I have a tentative fix opened in #58 since cargo is currently invoking a subcommand that doesn't have a version (hinted by the extra -sweep
)
--version
is working, and here's the --time
wrong usage message:
$ cargo sweep -t
error: a value is required for '--time <TIME>' but none was supplied
And the help message.
$ cargo sweep --help
...
-t, --time <TIME>
Number of days backwards to keep
So this issue is solved (ping @jyn514).
Also it appears that the
--version
flag doesn't work:I installed with
cargo install
minutes ago so I have version0.5.0
.