Skip to content

Commit 229b9d7

Browse files
(PUP-11767) Fix EvenOdd violations and remove from rubocop_todo
1 parent 61101ce commit 229b9d7

File tree

3 files changed

+2
-8
lines changed

3 files changed

+2
-8
lines changed

.rubocop_todo.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -459,12 +459,6 @@ Style/Encoding:
459459
- 'lib/puppet/type/package.rb'
460460
- 'lib/puppet/util/windows/service.rb'
461461

462-
# This cop supports safe auto-correction (--auto-correct).
463-
Style/EvenOdd:
464-
Exclude:
465-
- 'lib/puppet/pops/serialization/json.rb'
466-
- 'lib/puppet/pops/types/types.rb'
467-
468462
# This cop supports safe auto-correction (--auto-correct).
469463
Style/ExpandPathArguments:
470464
Exclude:

lib/puppet/pops/serialization/json.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ def write_delim
182182
@nested.pop
183183
write_delim
184184
when Integer
185-
if (cnt % 2) == 0 || !nesting[0]
185+
if cnt.even? || !nesting[0]
186186
@io << ','
187187
else
188188
@io << ':'

lib/puppet/pops/types/types.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2841,7 +2841,7 @@ def from_array(from)
28412841
if from.size == 0
28422842
{}
28432843
else
2844-
unless from.size % 2 == 0
2844+
unless from.size.even?
28452845
raise TypeConversionError.new(_('odd number of arguments for Hash'))
28462846
end
28472847

0 commit comments

Comments
 (0)