myty commented 3 years ago

[x] Related GitHub issue(s) linked in PR description
[x] Destination branch merged, built and tested with your changes
[x] Code formatted and follows best practices and patterns
[x] Code builds cleanly (no additional warnings or errors)
[x] Manually tested
[x] Automated tests are passing
[x] No decreases in automated test coverage
[x] Documentation updated (readme, docs, comments, etc.)
[x] Localization: No hard-coded error messages in code files (minimally in string constants)

codecov[bot] commented 3 years ago

Codecov Report

Merging #80 (a4548a5) into main (a8e1b5b) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #80   +/-   ##
=======================================
  Coverage   95.25%   95.25%           
=======================================
  Files          17       17           
  Lines         295      295           
  Branches       31       31           
=======================================
  Hits          281      281           
  Misses         13       13           
  Partials        1        1

Impacted Files	Coverage Δ
src/tests/factories/factory-type.ts	`100.00% <ø> (ø)`
src/tests/factories/index.ts	`100.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 a8e1b5b...a4548a5. Read the comment docs.

brandongregoryscott commented 3 years ago

I forgot to mention - all of these will need to be exported from index.ts for consumer usage 👍

rsm-hcd / AndcultureCode.JavaScript.React

Network connection provider and hook #80

Codecov Report