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

Added some logs #353

Merged
merged 4 commits into from
Feb 28, 2019
Merged

Conversation

MarcoRossignoli
Copy link
Collaborator

@MarcoRossignoli MarcoRossignoli commented Feb 27, 2019

Added some useful logs, filters, hits file, to show:

... -verbosity:minimal

On my local I see issue with missing hits file, template tests.
image

@MarcoRossignoli MarcoRossignoli changed the title Add some logs Added some logs Feb 27, 2019
@codecov
Copy link

codecov bot commented Feb 27, 2019

Codecov Report

Merging #353 into master will increase coverage by 0.04%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##           master     #353      +/-   ##
==========================================
+ Coverage   86.99%   87.04%   +0.04%     
==========================================
  Files          31       31              
  Lines        3092     3102      +10     
==========================================
+ Hits         2690     2700      +10     
  Misses        402      402

@tonerdo
Copy link
Collaborator

tonerdo commented Feb 28, 2019

Thanks @MarcoRossignoli, what do you think about automatically pre-pending [coverlet] to all verbose logs to it's easy to pick them out?

@MarcoRossignoli
Copy link
Collaborator Author

Thanks @MarcoRossignoli, what do you think about automatically pre-pending [coverlet] to all verbose logs to it's easy to pick them out?

I thought about that...and I agree...I'll update ASAP

@MarcoRossignoli
Copy link
Collaborator Author

@tonerdo PTAL more clear!
image

@tonerdo tonerdo merged commit bb0a20f into coverlet-coverage:master Feb 28, 2019
@MarcoRossignoli MarcoRossignoli deleted the addsomelogs branch February 28, 2019 11:34
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

Successfully merging this pull request may close these issues.

2 participants