Skip to content

Commit 35473cf

Browse files
committed
Merge pull request #1186 from bf4/empty_else
Enforce case requires else; allow else nil
2 parents e9a15a3 + 1d8a3d0 commit 35473cf

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

.rubocop.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,13 @@ Style/ClassAndModuleChildren:
4545
Style/Documentation:
4646
Enabled: false
4747

48+
Style/MissingElse:
49+
Enabled: true
50+
EnforcedStyle: case
51+
52+
Style/EmptyElse:
53+
EnforcedStyle: empty
54+
4855
Style/MultilineOperationIndentation:
4956
EnforcedStyle: indented
5057

lib/active_model/serializer/include_tree.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,8 @@ def [](key)
102102
self.class.new(@hash[:*])
103103
when @hash.key?(:**)
104104
self.class.new(:** => {})
105+
else
106+
nil
105107
end
106108
end
107109
end

0 commit comments

Comments
 (0)