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

[FINE] Use renamed methods #17282

Merged
merged 1 commit into from
Apr 11, 2018
Merged

[FINE] Use renamed methods #17282

merged 1 commit into from
Apr 11, 2018

Conversation

masayag
Copy link
Contributor

@masayag masayag commented Apr 11, 2018

Methods on master branch were renamed, therefore the backport failed.
This PR uses the methods names before the change on master branch.

The issue occurred by backporting ManageIQ/manageiq-providers-ovirt#224

Methods on master branch were renamed, therefore the backport failed.
This PR uses the methods names before the change on master branch.
@masayag
Copy link
Contributor Author

masayag commented Apr 11, 2018

@simaishi this PR is required to fix the broken FINE branch.

@masayag
Copy link
Contributor Author

masayag commented Apr 11, 2018

@pkliczewski please review

@miq-bot
Copy link
Member

miq-bot commented Apr 11, 2018

Checked commit masayag@92bbe1c with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

@simaishi
Copy link
Contributor

The Travis failures are unrelated to this PR, merging.

@simaishi simaishi merged commit ad8f735 into ManageIQ:fine Apr 11, 2018
@simaishi simaishi added this to the Sprint 84 Ending Apr 23, 2018 milestone Apr 11, 2018
@simaishi simaishi added the bug label Apr 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants