Skip to content

fix bug #769 Get Snapshots commands returns snapshots in random order#797

Merged
GilGald merged 3 commits intodevelopfrom
feature/apiddubny_bug_769_get_snapshots_command_returns_in_random_order
Sep 20, 2016
Merged

fix bug #769 Get Snapshots commands returns snapshots in random order#797
GilGald merged 3 commits intodevelopfrom
feature/apiddubny_bug_769_get_snapshots_command_returns_in_random_order

Conversation

@anthony-poddubny
Copy link
Copy Markdown

@anthony-poddubny anthony-poddubny commented Sep 19, 2016

Description

  • save snapshots in the OrderedDict instead of simple dictionary

Breaking

NO


This change is Reviewable

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 19, 2016

Coverage Status

Coverage increased (+0.005%) to 85.89% when pulling 75ce257 on feature/apiddubny_bug_769_get_snapshots_command_returns_in_random_order into 9575a50 on develop.

@GilGald
Copy link
Copy Markdown
Contributor

GilGald commented Sep 19, 2016

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from Reviewable

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 19, 2016

Coverage Status

Coverage increased (+0.005%) to 85.83% when pulling 0d2f665 on feature/apiddubny_bug_769_get_snapshots_command_returns_in_random_order into 7960587 on develop.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-72.8%) to 13.023% when pulling feff7de on feature/apiddubny_bug_769_get_snapshots_command_returns_in_random_order into 5d6efe0 on develop.

@GilGald GilGald merged commit d42964e into develop Sep 20, 2016
@GilGald GilGald removed the ready label Sep 20, 2016
@anthony-poddubny anthony-poddubny deleted the feature/apiddubny_bug_769_get_snapshots_command_returns_in_random_order branch October 12, 2016 15:22
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.

4 participants