Skip to content

Conversation

dpickle2
Copy link

@dpickle2 dpickle2 commented Jun 6, 2017

[This pull request is being opened now for review, but it should not be merged in yet as some work is still pending on the backend for duplication features to be fully supported.]

@storagedaemon
Copy link
Contributor

It seems like it'd be a good idea to merge this into #830 maybe?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 83.693% when pulling 84cea12 on dpickle2:feature/volume-detail-duplicate-info into 44efd83 on softlayer:master.

@dpickle2
Copy link
Author

dpickle2 commented Jun 6, 2017

I'm happy to merge this into #829 if desired. Should I go ahead and do that?

@storagedaemon
Copy link
Contributor

I'm thinking so; while they aren't strictly dependent on each other, one shouldn't be released without the other, so combining them would make sense.

@dpickle2
Copy link
Author

dpickle2 commented Jun 6, 2017

I have merged the commit into the feature/volume-duplicate branch, so these changes are now included in #830 .

@dpickle2 dpickle2 closed this Jun 6, 2017
@dpickle2 dpickle2 deleted the feature/volume-detail-duplicate-info branch June 6, 2017 20:51
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