Make missing kwargs `nil` instead of `false` #53

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

kddnewton merged 1 commit into ruby-syntax-tree:main from vinistock:make_missing_kwargs_nil

Apr 20, 2022

Contributor

vinistock commented Apr 20, 2022

If one of the keyword arguments is false, the Visitor class fails to visit Params here because node is false instead of nil.


          Make missing kwargs nil instead of false

5b00be4

kddnewton merged commit 36f37fe into ruby-syntax-tree:main

vinistock deleted the make_missing_kwargs_nil branch

April 20, 2022 19:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet