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

[Gemfile] Lock the config gem #20882

Merged
merged 1 commit into from
Dec 9, 2020
Merged

Conversation

NickLaMuro
Copy link
Member

@NickLaMuro NickLaMuro commented Dec 9, 2020

The config gem had a recent change to it that was recently released which broke our build:

So for now, lock the version to a known working version for us.

Links

@miq-bot
Copy link
Member

miq-bot commented Dec 9, 2020

Checked commit NickLaMuro@288fd13 with ruby 2.6.3, rubocop 0.82.0, haml-lint 0.35.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@Fryguy Fryguy merged commit 5c3b3b2 into ManageIQ:master Dec 9, 2020
simaishi pushed a commit that referenced this pull request Dec 9, 2020
[Gemfile] Lock the config gem

(cherry picked from commit 5c3b3b2)
@simaishi
Copy link
Contributor

simaishi commented Dec 9, 2020

Kasparov backport details:

$ git log -1
commit 8eb7ded37fdc87a63749a4293585ef539ccdbc49
Author: Jason Frey <fryguy9@gmail.com>
Date:   Wed Dec 9 13:51:36 2020 -0500

    Merge pull request #20882 from NickLaMuro/lock-config

    [Gemfile] Lock the config gem

    (cherry picked from commit 5c3b3b208439b1bf0fc3dca19b96a6c1999bb4f3)

simaishi pushed a commit that referenced this pull request Dec 9, 2020
[Gemfile] Lock the config gem

(cherry picked from commit 5c3b3b2)
@simaishi
Copy link
Contributor

simaishi commented Dec 9, 2020

Jansa backport details:

$ git log -1
commit dd4ccea7b2ba9f525954d0885b92c7a566aaa949
Author: Jason Frey <fryguy9@gmail.com>
Date:   Wed Dec 9 13:51:36 2020 -0500

    Merge pull request #20882 from NickLaMuro/lock-config

    [Gemfile] Lock the config gem

    (cherry picked from commit 5c3b3b208439b1bf0fc3dca19b96a6c1999bb4f3)

@pkuczynski
Copy link

New version release with reverted PR introducing the issue: https://github.com/rubyconfig/config/releases/tag/2.2.3

@Fryguy Fryguy mentioned this pull request Dec 10, 2020
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