Skip to content

Commit 36f37fe

Browse files
authored
Merge pull request #53 from vinistock/make_missing_kwargs_nil
Make missing kwargs `nil` instead of `false`
2 parents 1f713a9 + 5b00be4 commit 36f37fe

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

lib/syntax_tree/parser.rb

+5-1
Original file line numberDiff line numberDiff line change
@@ -2079,12 +2079,16 @@ def on_params(
20792079
keyword_rest,
20802080
block
20812081
)
2082+
# This is to make it so that required keyword arguments
2083+
# have a `nil` for the value instead of a `false`.
2084+
keywords&.map! { |(key, value)| [key, value || nil] }
2085+
20822086
parts = [
20832087
*requireds,
20842088
*optionals&.flatten(1),
20852089
rest,
20862090
*posts,
2087-
*keywords&.flat_map { |(key, value)| [key, value || nil] },
2091+
*keywords&.flatten(1),
20882092
(keyword_rest if keyword_rest != :nil),
20892093
(block if block != :&)
20902094
].compact

0 commit comments

Comments
 (0)