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

Add StorIOContentResolver parameter to get resolver #818

Merged
merged 4 commits into from
Sep 11, 2017

Conversation

nikitin-da
Copy link
Collaborator

The second part of #519

@nikitin-da nikitin-da added this to the v2.0.0 milestone Aug 22, 2017
@nikitin-da nikitin-da self-assigned this Aug 22, 2017
@codecov-io
Copy link

codecov-io commented Aug 22, 2017

Codecov Report

Merging #818 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #818   +/-   ##
=======================================
  Coverage   97.39%   97.39%           
=======================================
  Files          89       89           
  Lines        2692     2692           
  Branches      302      302           
=======================================
  Hits         2622     2622           
  Misses         38       38           
  Partials       32       32
Impacted Files Coverage Δ
...tentresolver/operations/get/PreparedGetCursor.java 93.33% <ø> (ø) ⬆️
...o2/contentresolver/operations/get/GetResolver.java 100% <ø> (ø) ⬆️
...tentresolver/operations/get/PreparedGetObject.java 100% <100%> (ø) ⬆️
...ver/operations/get/PreparedGetNumberOfResults.java 100% <100%> (ø) ⬆️
...olver/operations/get/PreparedGetListOfObjects.java 100% <100%> (ø) ⬆️

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 c02e1fc...25e10cb. Read the comment docs.

@nikitin-da nikitin-da merged commit bdd6f04 into master Sep 11, 2017
@nikitin-da nikitin-da deleted the issue-519-content-resolver branch September 11, 2017 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants