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 typo in get_conversion_log method #18136

Merged
merged 1 commit into from
Oct 26, 2018
Merged

Conversation

ghost
Copy link

@ghost ghost commented Oct 26, 2018

This PR fixes a typo that prevents downloading conversion log.

Associated RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1643561

@ghost
Copy link
Author

ghost commented Oct 26, 2018

@miq-bot add-label transformation, enhancement, hammer/yes
@miq-bot add-reviewer agrare

@miq-bot
Copy link
Member

miq-bot commented Oct 26, 2018

Checked commit fabiendupont@5013021 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@ghost
Copy link
Author

ghost commented Oct 26, 2018

@miq-bot remove-label enhancement
@miq-bot add-label bug

@miq-bot miq-bot added bug and removed enhancement labels Oct 26, 2018
@agrare agrare self-assigned this Oct 26, 2018
@agrare agrare merged commit 29907f5 into ManageIQ:master Oct 26, 2018
@agrare agrare added this to the Sprint 98 Ending Nov 5, 2018 milestone Oct 26, 2018
@ghost ghost deleted the bz1643561 branch October 26, 2018 15:19
simaishi pushed a commit that referenced this pull request Oct 26, 2018
@simaishi
Copy link
Contributor

Hammer backport details:

$ git log -1
commit 26956a0d43ca34fdde4e112aea23f0707d85167c
Author: Adam Grare <[email protected]>
Date:   Fri Oct 26 11:17:39 2018 -0400

    Merge pull request #18136 from fdupont-redhat/bz1643561
    
    Fix typo in get_conversion_log method
    
    (cherry picked from commit 29907f5cba436ee5af04d75580bab324418fe4b7)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1643561

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