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

Update MemoryStorage.*scan return value format #227

Merged
merged 3 commits into from
Jun 7, 2017

Conversation

scottinet
Copy link
Contributor

Description

Following the work done in kuzzleio/documentation#281, the returned values of MemoryStorage.*scan methods has been changed.

This PR updates the SDK to make it match the new return value format for these methods.

Note: this will trigger a patch version update on this SDK (5.0.2), as the published 5.x version is not usable until Kuzzle RC10 is out

@codecov-io
Copy link

codecov-io commented Jun 2, 2017

Codecov Report

Merging #227 into 5.x will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              5.x     #227      +/-   ##
==========================================
+ Coverage   98.24%   98.24%   +<.01%     
==========================================
  Files          17       17              
  Lines        2050     2052       +2     
  Branches      590      590              
==========================================
+ Hits         2014     2016       +2     
  Misses         36       36
Impacted Files Coverage Δ
src/MemoryStorage.js 99.03% <100%> (+0.01%) ⬆️

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 ec13b7f...0c406b1. Read the comment docs.

@@ -1,6 +1,6 @@
{
"name": "kuzzle-sdk",
"version": "5.0.1",
"version": "5.0.2",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought that package.json should be updated by the release PR, not there ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but as I stated last week, I plan to merge this PR directly on master once it is merged on 5.x, and to rewrite the tag to make the 5.x version of this SDK start at 5.0.2 instead of 5.0.0.
So I won't use the release tool and I want to merge this PR as-is, without modifying it before reaching master.

The idea is to avoid major versions jumps, as the 5.x version has to be released before Kuzzle RC10. It is currently unusable anyway, until RC10 is released.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@stafyniaksacha stafyniaksacha merged commit 0a1cc6f into 5.x Jun 7, 2017
@stafyniaksacha stafyniaksacha deleted the update-ms-scan-methods branch June 7, 2017 13:42
@scottinet scottinet mentioned this pull request Jun 20, 2017
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.

6 participants