urlstechie / urlchecker-python

:snake: :link: Python module and client for checking URLs
https://urlchecker-python.readthedocs.io
MIT License
20 stars 12 forks source link

option to only print red URLs? #29

Open maelle opened 4 years ago

maelle commented 4 years ago

Could there be an option to only print the URLs for which there was a failure? This way it'd be easier to go through them for a website that has many URLs?

vsoch commented 4 years ago

They are all printed in summary at the end, but yes we could definitely have a print fails only option!

SuperKogito commented 4 years ago

I think a better option here is to set some sort of print levels, or better print style something --prints-style that can take value like all, fails-only, maybe success-only too?

vsoch commented 4 years ago

@SuperKogito I think this idea is spot on - after we review and merge https://github.com/urlstechie/urlchecker-python/pull/28 would you care to work on this for urlchecker-python? I think the variable might be something like --print-level to mirror the idea of logging levels.

vsoch commented 4 years ago

I'm going to transfer this issue to urlchecker-python, as it needs to be implemented there. As a follow up it will need to be added here.

vsoch commented 4 years ago

This issue has been transferred to urlchecker-python - it will be followed up with update to urlchecker-action.

vsoch commented 4 years ago

@SuperKogito you should be unblocked here now too!

SuperKogito commented 4 years ago

yes I will make a PR of this today. Just wanted to ask, how about including a --print-level=none. The idea behind this, is that it can be useful, if we want to implement multiprocessing to provide a fast processing option, where the user will be only interested in the output file.