Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Issue/3835 #3836

Merged
merged 3 commits into from
May 22, 2017
Merged

Issue/3835 #3836

merged 3 commits into from
May 22, 2017

Conversation

flovilmart
Copy link
Contributor

@flovilmart flovilmart commented May 19, 2017

Fixes #3835

@flovilmart flovilmart requested a review from acinader May 19, 2017 20:55
@codecov
Copy link

codecov bot commented May 19, 2017

Codecov Report

Merging #3836 into master will increase coverage by 0.04%.
The diff coverage is 96%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3836      +/-   ##
==========================================
+ Coverage   90.15%   90.19%   +0.04%     
==========================================
  Files         114      114              
  Lines        7534     7550      +16     
==========================================
+ Hits         6792     6810      +18     
+ Misses        742      740       -2
Impacted Files Coverage Δ
src/Controllers/DatabaseController.js 94.34% <96%> (-0.02%) ⬇️
...dapters/Storage/Postgres/PostgresStorageAdapter.js 95.46% <0%> (+0.13%) ⬆️
src/Adapters/Cache/InMemoryCache.js 100% <0%> (+7.69%) ⬆️

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 03b6449...6e4dc98. Read the comment docs.

@flovilmart
Copy link
Contributor Author

@acinader could you have a look? This look quite urgent.

@flovilmart flovilmart closed this May 22, 2017
@flovilmart flovilmart reopened this May 22, 2017
@flovilmart flovilmart merged commit b5a2042 into master May 22, 2017
@flovilmart flovilmart deleted the issue/3835 branch May 22, 2017 16:34
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.

None yet

2 participants