-
Notifications
You must be signed in to change notification settings - Fork 23
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 #93
Conversation
inventory_refresh.gemspec
Outdated
@@ -23,7 +23,7 @@ Gem::Specification.new do |spec| | |||
spec.require_paths = ["lib"] | |||
|
|||
spec.add_dependency "activerecord", "~> 5.0" | |||
spec.add_dependency "more_core_extensions", "~> 3.5" | |||
spec.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.
It looks like ">=3.5", "< 5"
is safe here.
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@3669060 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
@agrare Can you review/merge and release a patch version?
Yes, MIQ uses an older version (v0.2) so we'll need to cherry-pick this back to that branch as well |
bump more core extensions (cherry picked from commit 1caa980)
Changes: * Bump more core extensions (#93)
Release v0.3.6 and v0.2.2 with this change |
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