-
Notifications
You must be signed in to change notification settings - Fork 356
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
bump more core extensions #7154
Conversation
manageiq-ui-classic.gemspec
Outdated
@@ -24,7 +24,7 @@ Gem::Specification.new do |s| | |||
s.add_dependency "coffee-rails" | |||
s.add_dependency "font-fabulous", "~> 1.0.5" | |||
s.add_dependency "high_voltage", "~> 3.0.0" | |||
s.add_dependency "more_core_extensions", "~>3.2" | |||
s.add_dependency "more_core_extensions", "~>4.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this can also become ">= 3.2", "< 5"
more core ext's on 4.1.0 update for update sake https://github.com/ManageIQ/more_core_extensions/compare/179bf40..e5b4501 - Added Ruby 2.7 support [[ManageIQ#79](ManageIQ/more_core_extensions#79)] - Added Process#pause, Process#resume, and Process#alive? [[ManageIQ#73](ManageIQ/more_core_extensions#73)] array added * `#compact_map` - Collect non-nil results from the block array added `#tabular_sort` - Sorts an Array of Hashes by specific columns hierarchy added `#descendant_get` - Returns the descendant with a given name the two breaking changes: - **BREAKING**: Moved Object#descendant_get to Class#descendant_get [[ManageIQ#75](ManageIQ/more_core_extensions#75)] - **BREAKING**: Removed deprecated Enumerable#stable_sort_by [[ManageIQ#76](ManageIQ/more_core_extensions#76)] a minor header output change was made that hasn't been released yet to make tableize more markdown compliant see ManageIQ/linux_admin#221
Checked commit d-m-u@642dfca with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LGTM
@h-kataria Can you review / merge?
more core ext's on 4.1.0
update for update sake
https://github.com/ManageIQ/more_core_extensions/compare/179bf40..e5b4501
array added *
#compact_map
- Collect non-nil results from the blockarray added
#tabular_sort
- Sorts an Array of Hashes by specific columnshierarchy added
#descendant_get
- Returns the descendant with a given namethe two breaking changes:
a minor header output change was made that hasn't been released yet to make tableize more markdown compliant
see ManageIQ/linux_admin#221