Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add the option for a short report #502

Merged
merged 4 commits into from
Mar 19, 2022
Merged

Conversation

goar5670
Copy link
Contributor

Description

Thanks for contributing this Pull Request. Add a brief description of what this Pull Request does. Do tag the relevant issue(s) and PR(s) below. If required, add some screenshot(s) to support your changes.

@codecov-commenter
Copy link

codecov-commenter commented Mar 18, 2022

Codecov Report

Merging #502 (e9166e8) into master (234815d) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #502      +/-   ##
==========================================
+ Coverage   89.18%   89.26%   +0.07%     
==========================================
  Files           8        8              
  Lines         555      559       +4     
==========================================
+ Hits          495      499       +4     
  Misses         60       60              
Impacted Files Coverage Δ
lib/colorls/core.rb 90.08% <100.00%> (+0.08%) ⬆️
lib/colorls/flags.rb 88.23% <100.00%> (+0.15%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 234815d...e9166e8. Read the comment docs.

Copy link
Collaborator

@avdv avdv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@avdv avdv merged commit 1f2c7ee into athityakumar:master Mar 19, 2022
@avdv
Copy link
Collaborator

avdv commented Mar 19, 2022

Thank you @goar5670 !

@goar5670 goar5670 deleted the short-report branch March 19, 2022 11:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants