Skip to content

Add jumpSearch and Tests #78

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

Merged
merged 2 commits into from
Oct 13, 2019
Merged

Add jumpSearch and Tests #78

merged 2 commits into from
Oct 13, 2019

Conversation

rubysdeadname
Copy link
Contributor

implemented time complextiy

implemented time complextiy
@TheSTL TheSTL mentioned this pull request Oct 12, 2019
@TheSTL
Copy link
Member

TheSTL commented Oct 12, 2019

Can you comment about time complexity ?
You can take example of this PR - [#76 ]

@rubysdeadname
Copy link
Contributor Author

rubysdeadname commented Oct 12, 2019

@TheSTL Sorry for the confusion, I'm new to contributing to open source. Hopefully this is all ok. Thanks for the opportunity to contribute!

Copy link
Member

@TheSTL TheSTL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gabrielbarker . You implemented it nicely 👍

@TheSTL TheSTL merged commit d59cc09 into knaxus:master Oct 13, 2019
@TheSTL TheSTL mentioned this pull request Oct 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants