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.
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
Optimize find method #226
Optimize find method #226
Conversation
depends on #224 |
There is one thing that worries me. If someone can explain under what conditions this code can be executed, I will be very grateful. Because I think that this is an unreachable path of the code and tests also don't cover it. Line 508 in 9207236
|
Good job!!!! |
471a55f
to
7d905af
Compare
7d905af
to
aa27eb4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heya! The changes looks good, but I would like to take a little more time to review them! I'm currently on PTO, I'll take a look as soon as I'm back :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
I refactored the find method. The main things that were changed: