Skip to content
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

Clean up HappyMapper::ClassMethods#find_nodes_to_parse #242

Merged
merged 1 commit into from
Nov 10, 2024
Merged

Clean up HappyMapper::ClassMethods#find_nodes_to_parse #242

merged 1 commit into from
Nov 10, 2024

Commits on Nov 9, 2024

  1. Clean up HappyMapper::ClassMethods#find_nodes_to_parse 

    This complex code triggered a RuboCop offense, and going by the specced
behavior, could be made a lot simpler.

The value of options[:name] was never actually used to find nodes, and
also makes less sense because it is used to set the attribute name in
the containing class.
    @mvz
    mvz committed Nov 9, 2024
    Configuration menu
    Copy the full SHA
    c1a0428 View commit details
    Browse the repository at this point in the history