Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Luni-4
Copy link
Collaborator

@Luni-4 Luni-4 commented Nov 16, 2020

Html language wasn't used for any kind of metric, so it is better to remove it

marco-c
marco-c previously approved these changes Nov 16, 2020
@codecov-io
Copy link

codecov-io commented Nov 16, 2020

Codecov Report

Merging #360 (ced8e91) into master (d893d07) will increase coverage by 0.23%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #360      +/-   ##
==========================================
+ Coverage   22.35%   22.58%   +0.23%     
==========================================
  Files          46       45       -1     
  Lines        5722     5663      -59     
  Branches      813      808       -5     
==========================================
  Hits         1279     1279              
+ Misses       3913     3854      -59     
  Partials      530      530              
Impacted Files Coverage Δ
src/alterator.rs 32.46% <ø> (+4.68%) ⬆️
src/checker.rs 36.36% <ø> (ø)
src/getter.rs 48.48% <ø> (ø)
src/langs.rs 90.00% <ø> (ø)
src/metrics/cognitive.rs 51.59% <ø> (ø)
src/metrics/cyclomatic.rs 56.96% <ø> (ø)
src/metrics/exit.rs 58.82% <ø> (ø)
src/metrics/fn_args.rs 64.40% <ø> (ø)
src/metrics/halstead.rs 48.38% <ø> (ø)
src/metrics/loc.rs 58.73% <ø> (ø)
... and 4 more

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 d893d07...ced8e91. Read the comment docs.

@marco-c marco-c merged commit cb37ad9 into mozilla:master Nov 20, 2020
@Luni-4 Luni-4 deleted the remove-html branch November 20, 2020 10:40
spacedragon pushed a commit to spacedragon/rust-code-analysis that referenced this pull request Feb 2, 2021
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