Skip to content
This repository has been archived by the owner on Mar 5, 2018. It is now read-only.

[TFF-41] Faults story #97

Merged
merged 17 commits into from
Mar 30, 2017
Merged

[TFF-41] Faults story #97

merged 17 commits into from
Mar 30, 2017

Conversation

trevinhofmann
Copy link
Member

@trevinhofmann trevinhofmann commented Mar 30, 2017

Summary

We can log faults and remove them.

Checklist

  • Have you added unit and functional tests as appropriate?
  • Did you update/add documentation for new methods or changed functionality?
  • Have you merged the target branch down into this one?

How to Review

  1. Run with the sister PR.
  2. Make sure it works.

Links

@codecov-io
Copy link

codecov-io commented Mar 30, 2017

Codecov Report

Merging #97 into dev will decrease coverage by 0.01%.
The diff coverage is 87.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev      #97      +/-   ##
==========================================
- Coverage   88.19%   88.17%   -0.02%     
==========================================
  Files          20       20              
  Lines         559      575      +16     
==========================================
+ Hits          493      507      +14     
- Misses         66       68       +2
Impacted Files Coverage Δ
src/store/item-store.js 96.36% <100%> (+0.36%) ⬆️
src/api/item.js 71.42% <81.81%> (+4.76%) ⬆️

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 d00fcfd...0f60535. Read the comment docs.

@trevinhofmann trevinhofmann merged commit 5842e4a into dev Mar 30, 2017
@tondef001 tondef001 deleted the faults-tff41-story branch March 30, 2017 06:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants