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

[DARGA] Backport #9667 Fix gem issues related to rails 5.0 release #9756

Merged
merged 3 commits into from Jul 13, 2016

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Jul 12, 2016

Rails 5 has a minimum Ruby version of 2.2.2. Several of our gems depend on ActiveSupport. In order to support older Ruby versions in our gems, I've added blank to more_core_extensions and dropped the dependencies on activesupport.

Backport of #9667

@bdunne
Copy link
Member Author

bdunne commented Jul 12, 2016

@chessbyte Please review and label

@Fryguy Fryguy changed the title Backport #9667 Fix gem issues related to rails 5.0 release [Darga] Backport #9667 Fix gem issues related to rails 5.0 release Jul 12, 2016
@chessbyte chessbyte changed the title [Darga] Backport #9667 Fix gem issues related to rails 5.0 release [DARGA] Backport #9667 Fix gem issues related to rails 5.0 release Jul 12, 2016
@chessbyte chessbyte self-assigned this Jul 12, 2016
@miq-bot
Copy link
Member

miq-bot commented Jul 12, 2016

Checked commits bdunne/manageiq@549725f~...4927a1b with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
3 files checked, 1 offense detected

gems/pending/util/miq-system.rb

@chessbyte chessbyte closed this Jul 12, 2016
@chessbyte chessbyte reopened this Jul 12, 2016
@chessbyte
Copy link
Member

@bdunne can you review failures on Travis

@bdunne
Copy link
Member Author

bdunne commented Jul 12, 2016

@chessbyte looks like those failures are caused by 6ff4157
https://travis-ci.org/ManageIQ/manageiq/builds/144280631

@chessbyte chessbyte closed this Jul 13, 2016
@chessbyte chessbyte reopened this Jul 13, 2016
@chessbyte chessbyte closed this Jul 13, 2016
@chessbyte chessbyte reopened this Jul 13, 2016
@chessbyte chessbyte merged commit 5f3c47d into ManageIQ:darga Jul 13, 2016
@chessbyte chessbyte added this to the Sprint 44 Ending Aug 1, 2016 milestone Jul 13, 2016
@bdunne bdunne deleted the darga_upgrade_gems branch July 13, 2016 19:39
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.

None yet

3 participants