-
Notifications
You must be signed in to change notification settings - Fork 61
Add a --no-readme flag to time command
#57
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
Conversation
Maybe this is a good time to re-think how we approach the time command now that it is its own command.
The described behavior can be achieved by running cargo {solve|all} <day> --release --time, which is a lot of typing. Maybe istd. of introducing --no-readme, we can make --no-readme the default of time:
- Remove the
--timeflag fromsolveandall. Make this the default behavior oftime. - Add a
--storeflag totimeto update the JSON & readme.
What do you think?
Maybe this is a good time to re-think how we approach the
timecommand now that it is its own command.The described behavior can be achieved by running
cargo {solve|all} <day> --release --time, which is a lot of typing. Maybe istd. of introducing--no-readme, we can make--no-readmethe default oftime:
- Remove the
--timeflag fromsolveandall. Make this the default behavior oftime.- Add a
--storeflag totimeto update the JSON & readme.What do you think?
I agree it will avoid having duplicate command behaviour 👍
implemented in #58
Allow to not update readme table every time
timeis called.Especially when using
cargo time <day>multiple time in row.