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

Looks like self is needed on PRODUCT_NAME and VERSION calls #16556

Merged
merged 1 commit into from
Nov 29, 2017

Conversation

juliancheal
Copy link
Member

@juliancheal juliancheal commented Nov 29, 2017

Also making sure PRODUCT_NAME is also frozen.

In trying to get this Lenovo PR green ManageIQ/manageiq-providers-lenovo#96 we realised we should have kept the self. prefix to PRODUCT_NAME and VERSION in #16410

Otherwise you get

Vmdb::Appliance.USER_AGENT NameError: uninitialised constant Vmdb::Appliance::PRODUCT_NAME

/cc @Fryguy

Also making sure PRODUCT_NAME is also frozen.
@juliancheal
Copy link
Member Author

@miq-bot assign @Fryguy

@miq-bot
Copy link
Member

miq-bot commented Nov 29, 2017

@juliancheal Cannot apply the following label because they are not recognized: bug gaprindashvili/yes core

@juliancheal
Copy link
Member Author

@miq-bot add_label bug, gaprindashvili/yes, core

@miq-bot
Copy link
Member

miq-bot commented Nov 29, 2017

Checked commit juliancheal@3355662 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 assigned chessbyte and unassigned Fryguy Nov 29, 2017
@chessbyte chessbyte merged commit eaa0a19 into ManageIQ:master Nov 29, 2017
@chessbyte chessbyte added this to the Sprint 75 Ending Dec 11, 2017 milestone Nov 29, 2017
simaishi pushed a commit that referenced this pull request Nov 29, 2017
Looks like self is needed on PRODUCT_NAME and VERSION calls
(cherry picked from commit eaa0a19)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 781eb0254ce1939597139a4486d30dc8c52b1414
Author: Oleg Barenboim <chessbyte@gmail.com>
Date:   Wed Nov 29 13:35:02 2017 -0500

    Merge pull request #16556 from juliancheal/fix_user_agent
    
    Looks like self is needed on PRODUCT_NAME and VERSION calls
    (cherry picked from commit eaa0a19f41b23be232efda0f6af9afec85f79d9a)

@juliancheal
Copy link
Member Author

@simaishi Thanks 👍

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