Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Store filter mappings to improve future update performance #28

Merged
merged 2 commits into from
Mar 20, 2018

Conversation

diogofcunha
Copy link
Owner

Relates to comment on #23

@codecov-io
Copy link

codecov-io commented Mar 20, 2018

Codecov Report

Merging #28 into master will decrease coverage by 0.13%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
- Coverage   94.95%   94.82%   -0.14%     
==========================================
  Files          14       15       +1     
  Lines         119      116       -3     
==========================================
- Hits          113      110       -3     
  Misses          6        6
Impacted Files Coverage Δ
src/selectors/filtering.js 100% <100%> (ø)
src/FilteringContainer.js 94.11% <100%> (-1.72%) ⬇️

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 55183b2...aa2e426. Read the comment docs.

@coveralls
Copy link

coveralls commented Mar 20, 2018

Coverage Status

Coverage decreased (-0.08%) to 92.529% when pulling aa2e426 on filtering-revamp into 55183b2 on master.

@diogofcunha diogofcunha merged commit 4ebc80b into master Mar 20, 2018
@diogofcunha diogofcunha deleted the filtering-revamp branch March 20, 2018 13:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants