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

Replaced validate_* methods calls by supports_* for snapshot operations #437

Merged
merged 2 commits into from
Feb 22, 2017

Conversation

gauravaradhye
Copy link
Contributor

@gauravaradhye gauravaradhye commented Feb 20, 2017

This PR is related to the refactoring changes in PR ManageIQ/manageiq#13992

@gauravaradhye
Copy link
Contributor Author

@miq-bot add_label wip

@miq-bot miq-bot added the wip label Feb 20, 2017
@gauravaradhye gauravaradhye changed the title [WIP]Replaced validate_* methods calls by supports_* for snapshot operations Replaced validate_* methods calls by supports_* for snapshot operations Feb 21, 2017
@gauravaradhye
Copy link
Contributor Author

@miq-bot remove_label wip

@gauravaradhye
Copy link
Contributor Author

@miq-bot assign @durandom

@miq-bot
Copy link
Member

miq-bot commented Feb 21, 2017

@gauravaradhye @durandom is an invalid assignee, ignoring...

@miq-bot
Copy link
Member

miq-bot commented Feb 21, 2017

Checked commits gauravaradhye/manageiq-ui-classic@3717205~...b9d8d60 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 👍

Copy link
Member

@durandom durandom left a comment

Choose a reason for hiding this comment

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

LGTM
should turn green when the core PR is merged

@martinpovolny
Copy link
Member

Hmm, well it did not. The error message is a different one.

Is that change on purpose?

Did anyone actually run the specs?

@durandom
Copy link
Member

Did anyone actually run the specs?

apparently not 😭 my fault
I fixed required changes to core in ManageIQ/manageiq#14014
Then we can merge #444 in favor of this

@martinpovolny martinpovolny merged commit b9d8d60 into ManageIQ:master Feb 22, 2017
@chessbyte chessbyte added this to the Sprint 55 Ending Feb 27, 2017 milestone Mar 17, 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.

5 participants