Skip to content
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

Denoise show-all output #484

Open
eugenesvk opened this issue Jul 25, 2022 · 1 comment
Open

Denoise show-all output #484

eugenesvk opened this issue Jul 25, 2022 · 1 comment

Comments

@eugenesvk
Copy link

Currently the detailed output of cargo crev v --show-all is too noisy mostly because it includes many unneeded numbers due to e.g. unnecessary precision (121015 is less informative than .1m) or 0s

Below are a few suggestions re. how to make it look nicer and more informative:

  • replace 0 with spaces
  • change # downloads to whole millions (only show the fractional part if below .5m and show a max of one fractional digit, so 10345 would be .0)
  • change locs to thousands
  • add a sub-header to add unit dimension (K and M) and totals (or maybe a simple + if breaks alignment with some fonts)

The ouput with these suggestions implemented would look like following

===AFTER===
status reviews issues owner downloads   loc lpidx geiger flgs
             ∑     ∑      ∑ M       ∑   K
none         1            2   10   48    .3   161     32 CB__
pass     2   4            2   40   43    .1   381        ____
none         9            1    5   89    .2   249      7 ____
none         2            3    .7  20    .2   132        CB__
none        12            2    1   21   3.4    32   1201 ____
none                      2   10   46    .5   122    112 CB__
pass     3   8            2   72   94    .1   297      8 ____
===BEFORE===
status reviews issues owner      downloads    loc lpidx geiger flgs
none     0   1  0   0  0  2 10125K  47840K    323   161     32 CB__
pass     2   4  0   0  0  2 39838K  42837K     51   381      0 ____
none     0   9  0   0  0  1  5182K  89435K    252   249      7 ____
none     0   2  0   0  0  3   690K  20454K    202   132      0 CB__
none     0  12  0   0  0  2   979K  20714K   3463    32   1201 ____
none     0   0  0   0  0  2  9620K  46298K    529   122    112 CB__
pass     3   8  0   0  0  2 72455K  94052K    134   297      8 ____
  • Please let us know:
    • cargo-crev 0.23.2
    • installed by downloading this binary from your release page and moved it to ...cargo\bin
    • Windows 10
@dpc
Copy link
Collaborator

dpc commented Jul 25, 2022

👍


pub fn print_header(

for anyone that wants to give it a chance.

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

No branches or pull requests

2 participants