Skip to content

Conversation

@marco-ballario
Copy link
Contributor

This PR adds some missing checks on the LLOC metric in some unit tests for the LOC metrics. Thanks in advance for the review.

@codecov-commenter
Copy link

codecov-commenter commented Dec 30, 2021

Codecov Report

Merging #739 (210bf85) into master (f9c1454) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #739   +/-   ##
=======================================
  Coverage   37.95%   37.95%           
=======================================
  Files          51       51           
  Lines        6549     6549           
  Branches      963      963           
=======================================
  Hits         2486     2486           
  Misses       3403     3403           
  Partials      660      660           
Impacted Files Coverage Δ
src/metrics/loc.rs 77.37% <ø> (ø)

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 f9c1454...210bf85. Read the comment docs.

@marco-ballario marco-ballario force-pushed the more-lloc-checks branch 3 times, most recently from d50775e to 210bf85 Compare December 30, 2021 17:32
@marco-c marco-c merged commit 736f379 into mozilla:master Jan 10, 2022
@marco-ballario marco-ballario deleted the more-lloc-checks branch February 9, 2022 13:50
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.

3 participants