Skip to content

Conversation

tguichaoua
Copy link
Contributor

Allow to not update readme table every time time is called.
Especially when using cargo time <day> multiple time in row.

@fspoettel
Copy link
Owner

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:

  1. Remove the --time flag from solve and all. Make this the default behavior of time.
  2. Add a --store flag to time to update the JSON & readme.

What do you think?

@tguichaoua
Copy link
Contributor Author

tguichaoua commented Dec 11, 2023

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:

  1. Remove the --time flag from solve and all. Make this the default behavior of time.
  2. Add a --store flag to time to update the JSON & readme.

What do you think?

I agree it will avoid having duplicate command behaviour 👍

@tguichaoua tguichaoua closed this Dec 11, 2023
@fspoettel
Copy link
Owner

implemented in #58

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 this pull request may close these issues.

2 participants