Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Adds respond_to_missing? in classes where method_missing is implemented. #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adds respond_to_missing? in classes where method_missing is implemented. #65

wants to merge 1 commit into from

Conversation

shaunakpp
Copy link
Contributor

@shaunakpp shaunakpp commented Oct 11, 2018 •
edited
Loading

  • Also uses guard clause instead of deep conditional nesting in scope.rb method_missing
  • Also adds 'super' check in method_missing so that it doesn't blow on actual missing methods

This change is Reviewable

@shaunakpp
Adds respond_to_missing? in classes where method_missing is implemented.
493a653 
Also uses guard clause instead of deep conditional nesting in scope.rb method_missing
Also adds 'super' check in method_missing so that it doesn't blow on actual missing methods
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@shaunakpp