Skip to content

Commit 6de0f01

Browse files
chore(internal): minor refactoring of utils (#67)
1 parent 410b86e commit 6de0f01

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

Rakefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ CLEAN.push(*%w[.idea/ .ruby-lsp/ .yardoc/])
1111

1212
xargs = %w[xargs --no-run-if-empty --null --max-procs=0 --max-args=300 --]
1313

14-
task(default: [:test])
14+
multitask(default: [:test])
1515

1616
multitask(:test) do
1717
rb =

lib/openai/util.rb

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ def primitive?(input)
7575
def coerce_boolean(input)
7676
case input.is_a?(String) ? input.downcase : input
7777
in Numeric
78-
!input.zero?
78+
input.nonzero?
7979
in "true"
8080
true
8181
in "false"
@@ -165,14 +165,12 @@ class << self
165165
private def deep_merge_lr(lhs, rhs, concat: false)
166166
case [lhs, rhs, concat]
167167
in [Hash, Hash, _]
168-
# rubocop:disable Style/YodaCondition
169-
rhs_cleaned = rhs.reject { |_, val| OMIT == val }
168+
rhs_cleaned = rhs.reject { _2 == OpenAI::Util::OMIT }
170169
lhs
171-
.reject { |key, _| OMIT == rhs[key] }
170+
.reject { |key, _| rhs[key] == OpenAI::Util::OMIT }
172171
.merge(rhs_cleaned) do |_, old_val, new_val|
173172
deep_merge_lr(old_val, new_val, concat: concat)
174173
end
175-
# rubocop:enable Style/YodaCondition
176174
in [Array, Array, true]
177175
lhs.concat(rhs)
178176
else

0 commit comments

Comments
 (0)