Skip to content

Commit

Permalink
Also remove Hash#slice!
Browse files Browse the repository at this point in the history
  • Loading branch information
dduugg committed Dec 12, 2023
1 parent 4a062b1 commit 0ce8438
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 31 deletions.
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@
!**/vendor/bundle/ruby/*/gems/activesupport-*/lib/active_support/core_ext/hash/deep_merge.rb
!**/vendor/bundle/ruby/*/gems/activesupport-*/lib/active_support/core_ext/hash/deep_transform_values.rb
!**/vendor/bundle/ruby/*/gems/activesupport-*/lib/active_support/core_ext/hash/keys.rb
!**/vendor/bundle/ruby/*/gems/activesupport-*/lib/active_support/core_ext/hash/slice.rb
!**/vendor/bundle/ruby/*/gems/activesupport-*/lib/active_support/core_ext/object/deep_dup.rb
!**/vendor/bundle/ruby/*/gems/activesupport-*/lib/active_support/core_ext/object/duplicable.rb
!**/vendor/bundle/ruby/*/gems/activesupport-*/lib/active_support/core_ext/string/exclude.rb
Expand Down
4 changes: 1 addition & 3 deletions Library/Homebrew/dev-cmd/update-maintainers.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@
require "utils/github"
require "manpages"

require "active_support/core_ext/hash/slice"

module Homebrew
module_function

Expand Down Expand Up @@ -39,7 +37,7 @@ def update_maintainers

sentences = {}
members.each do |group, hash|
hash.slice!(*public_members)
hash.replace(hash.slice(*public_members))

Check warning on line 40 in Library/Homebrew/dev-cmd/update-maintainers.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/dev-cmd/update-maintainers.rb#L40

Added line #L40 was not covered by tests
hash.each { |login, name| hash[login] = "[#{name}](https://github.com/#{login})" }
sentences[group] = hash.values.sort.to_sentence
end
Expand Down

This file was deleted.

0 comments on commit 0ce8438

Please sign in to comment.