Skip to content

Conversation

wasabeef
Copy link
Member

What does this change?

  • Replace test_coverage, avoid test_coverage in deps

What is the value of this and can you measure success?

  • Pass CI

@wasabeef wasabeef requested a review from lcdsmao February 19, 2021 19:40
@codecov
Copy link

codecov bot commented Feb 19, 2021

Codecov Report

Merging #84 (3b6685d) into main (992fd75) will decrease coverage by 0.20%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #84      +/-   ##
==========================================
- Coverage   98.22%   98.01%   -0.21%     
==========================================
  Files          16       16              
  Lines         506      505       -1     
==========================================
- Hits          497      495       -2     
- Misses          9       10       +1     
Impacted Files Coverage Δ
packages/core/lib/utils/file.dart 75.00% <0.00%> (-25.00%) ⬇️
packages/core/lib/generators/assets_generator.dart 100.00% <0.00%> (ø)

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 992fd75...3b6685d. Read the comment docs.

@wasabeef wasabeef requested a review from lcdsmao February 20, 2021 06:00
@wasabeef wasabeef merged commit b4d31b3 into main Feb 20, 2021
@wasabeef wasabeef deleted the replace-test-coverage branch February 20, 2021 06:12
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