cs50 / check50

This is check50, a command-line program with which you can check the correctness of your programs.
GNU General Public License v3.0
399 stars 267 forks source link

remove --verbose #227

Closed cmlsharp closed 4 years ago

cmlsharp commented 4 years ago

It seems to me that we could replace --verbose with --log-level info. I'm almost postive this wouldn't change any behavior since at present --verbose does two things: display installed dependencies and stdout and stderr from running checks. Both of these only apply when running checks locally, meaning nothing from lib50 would even be in INFO log. We may as well just use our logging functionality for both of these.

Jelleas commented 4 years ago

You had me at the title, but this was the punchline:

only apply when running checks locally

To add, I can't think of a good reason you'd want to run

--local --log-level info

or

--local --verbose

But not

--local --log-level info --verbose

So indeed let's drop --verbose and go with --log-level info as its replacement.