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

[FEATURE REQUEST] octocov to show more information #374

Open
omuomugin opened this issue May 30, 2024 · 3 comments
Open

[FEATURE REQUEST] octocov to show more information #374

omuomugin opened this issue May 30, 2024 · 3 comments

Comments

@omuomugin
Copy link

omuomugin commented May 30, 2024

Hey thank you for the great tool.

Right now I heavily using octocov command in CLI usecase.

It might be more useful to have more information than overall coverage only.

For example Files, Lines, Covered which printed in detail for Comment and Summary in GitHub Actions usecase.

$ octocov -r path/to/report

                         path/to/report             
----------------------------------------
Coverage                      93.0% 
Files                         XXXX
Lines                         XXXX
Covered                       XXXX

and also are there any chance that I can change path/to/report to something more shorter ?

Anyway thanks for great tools you make and keep up the good works.

@omuomugin omuomugin changed the title [FEATURE REQUEST] octocov to show comment mode or summary mode [FEATURE REQUEST] octocov to show more information May 30, 2024
@k1LoW
Copy link
Owner

k1LoW commented May 30, 2024

@omuomugin Thank you for your request!

It might be more useful to have more information than overall coverage only.

Now, the detailed information function is split between octocov ls-files and octopov view [FILE_PATH].

are there any chance that I can change path/to/report to something more shorter ?

No arguments needed if .octocov.yml is located with the correct information.

In particular, it would be good to set coverage.paths: to.

Also, if set up correctly, the Code to Test Ratio can be checked locally.

For example, in the octocov repository, the following is shown.

~/src/github.com/k1LoW/octocov (main)> octocov

                      main (6ce51bb)
--------------------------------------
  Coverage                     50.2%
  Code to Test Ratio           1:0.6

~/src/github.com/k1LoW/octocov (main)>

@omuomugin
Copy link
Author

Now, the detailed information function is split between octocov ls-files and octopov view [FILE_PATH].

OK, I understand that.
My opinion was octocov ls-files and octopov view [FILE_PATH] are too detailed so just wanted to see summary in octocov.

No arguments needed if .octocov.yml is located with the correct information.

Thanks for further information, I'll try setting coverage.paths: too.

@k1LoW
Copy link
Owner

k1LoW commented May 31, 2024

so just wanted to see summary in octocov.

fmfm. I'll give it some consideration 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants