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

Fix version check in supports_admin_ui method #156

Merged
merged 1 commit into from
Nov 27, 2017

Conversation

vojtechszocs
Copy link
Contributor

@vojtechszocs vojtechszocs commented Nov 27, 2017

version_higher_than was renamed to version_at_least via another PR and I forgot to sync with code that implements the admin_ui feature 😞

Depends on #143

version_higher_than was renamed to version_at_least

Signed-off-by: Vojtech Szocs <[email protected]>
@vojtechszocs
Copy link
Contributor Author

cc @jhernand @masayag

@miq-bot
Copy link
Member

miq-bot commented Nov 27, 2017

Checked commit vojtechszocs@bac427f with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@chessbyte chessbyte added this to the Sprint 74 Ending Nov 27, 2017 milestone Nov 27, 2017
@chessbyte chessbyte merged commit 3c7ebee into ManageIQ:master Nov 27, 2017
@chessbyte chessbyte self-assigned this Nov 27, 2017
@masayag
Copy link
Contributor

masayag commented Nov 27, 2017

@Fryguy see comment #133 (comment)

Backporting this PR alone will cause gaprindashvili to break. If wishes to backport it, #143 must be backported as well, since the current PR depends on it.

We can also not backport neither of them. But the two are depended on each other.

simaishi pushed a commit that referenced this pull request Nov 27, 2017
Fix version check in supports_admin_ui method
(cherry picked from commit 3c7ebee)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 8ead7c3f090bd9a8341959b184900b1ac7856c20
Author: Oleg Barenboim <[email protected]>
Date:   Mon Nov 27 11:07:44 2017 -0500

    Merge pull request #156 from vojtechszocs/fix-version-check
    
    Fix version check in supports_admin_ui method
    (cherry picked from commit 3c7ebee01ac380b2cd2e013f6f1f98e6a6d87cc3)

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.

6 participants