search

Perfexionists / perun

Lightweight Performance Control System
GNU General Public License v3.0
16 stars 14 forks source link

Enhance the output of the Perun #174

Closed tfiedor closed 8 months ago

tfiedor commented 9 months ago

This PR:

  1. Enhances the output on the commandline,
  2. Remodels the way information is given to user,
  3. Extract styling and colours to isolate package.
codecov[bot] commented 9 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (f9620f0) 98.70% compared to head (433379c) 98.68%.

Files Patch % Lines
perun/logic/config_templates.py 85.71% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## devel #174 +/- ## ========================================== - Coverage 98.70% 98.68% -0.02% ========================================== Files 131 131 Lines 8485 8539 +54 ========================================== + Hits 8375 8427 +52 - Misses 110 112 +2 ``` | [Flag](https://app.codecov.io/gh/Perfexionists/perun/pull/174/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Perfexionists) | Coverage Δ | | |---|---|---| | [coverage-3.10](https://app.codecov.io/gh/Perfexionists/perun/pull/174/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Perfexionists) | `?` | | | [coverage-3.11](https://app.codecov.io/gh/Perfexionists/perun/pull/174/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Perfexionists) | `98.68% <99.72%> (+0.03%)` | :arrow_up: | | [coverage-3.9](https://app.codecov.io/gh/Perfexionists/perun/pull/174/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Perfexionists) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Perfexionists#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tfiedor commented 8 months ago

Preview of the new look:

output-rm output-add output-init output-collect output-fuzzing
tfiedor commented 8 months ago

Great, I like the new logging API and the way you can build the terminal output. I noticed the log messages regarding job progress in one of the screenshots, should we perhaps consider enhancing logging with some terminal progress bar (e.g., think of apt or dnf package managers)?

Good point, though not for the particular case, I can see the usage for the "ticks". I will merge this and make an issue out of this.