Skip to content

Conversation

hkethi002
Copy link
Contributor

Fixes #865

Review Checklist

  • Tests were added to cover all code changes
  • Documentation was added / updated
  • Code and tests follow standards in CONTRIBUTING.md

@hkethi002 hkethi002 requested a review from nagem September 25, 2017 17:11
@codecov-io
Copy link

codecov-io commented Sep 25, 2017

Codecov Report

Merging #939 into master will decrease coverage by 0.01%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #939      +/-   ##
==========================================
- Coverage   90.19%   90.18%   -0.02%     
==========================================
  Files          48       48              
  Lines        6418     6420       +2     
==========================================
+ Hits         5789     5790       +1     
- Misses        629      630       +1
Flag Coverage Δ
#python 90.18% <75%> (-0.02%) ⬇️
Impacted Files Coverage Δ
api/handlers/dataexplorerhandler.py 84.4% <75%> (-0.28%) ⬇️

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 124ea0c...b579da6. Read the comment docs.

@nagem
Copy link
Contributor

nagem commented Oct 11, 2017

Changes look good!

@hkethi002 hkethi002 merged commit 8954b48 into master Oct 11, 2017
@hkethi002 hkethi002 deleted the search-non-user branch October 11, 2017 22:03
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