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

Line ranges based on coverage #35

Merged
merged 3 commits into from
Sep 29, 2023
Merged

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Sep 29, 2023

When reporting consecutive lines that are not covered it makes sense to skip lines that have no recorded coverage information (blank lines, comments).

@uhafner uhafner added the enhancement Enhancement of existing functionality label Sep 29, 2023
@codecov
Copy link

codecov bot commented Sep 29, 2023

Codecov Report

Merging #35 (f24dd51) into main (fb6f111) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main      #35      +/-   ##
============================================
+ Coverage     95.05%   95.07%   +0.02%     
- Complexity      575      578       +3     
============================================
  Files            23       23              
  Lines          1394     1402       +8     
  Branches        197      201       +4     
============================================
+ Hits           1325     1333       +8     
  Misses           44       44              
  Partials         25       25              
Files Coverage Δ
src/main/java/edu/hm/hafner/coverage/FileNode.java 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@uhafner uhafner merged commit 83b21b1 into main Sep 29, 2023
30 checks passed
@uhafner uhafner deleted the line-ranges-based-on-coverage branch September 29, 2023 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant