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

Replace reflect.DeepEqual() with "github.com/google/go-cmp/cmp" #58

Merged
merged 2 commits into from
Oct 4, 2020

Conversation

infiniteoverflow
Copy link
Contributor

Issue URL : openebs/openebs#3143

@shubham14bajpai
Copy link
Contributor

@infiniteoverflow travis is failing due to import errors. Please add import for cmp and remove reflect.

Added import to "github.com/google/go-cmp/cmp" and removed reflect
@infiniteoverflow
Copy link
Contributor Author

@infiniteoverflow travis is failing due to import errors. Please add import for cmp and remove reflect.

@shubham14bajpai I have made the changes as requested.

Copy link

@prateekpandey14 prateekpandey14 left a comment

Choose a reason for hiding this comment

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

lgtm

@shubham14bajpai shubham14bajpai merged commit b2d33c8 into openebs-archive:master Oct 4, 2020
@shubham14bajpai
Copy link
Contributor

Thank you @infiniteoverflow for this PR. You can sign up for Hacktoberfest swag from MayaData by entering details here: https://github.com/mayadata-io/swag/issues/new/choose

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.

4 participants