Skip to content

Add jumpSearch algorithm #75

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
Closed
wants to merge 3 commits into from
Closed

Add jumpSearch algorithm #75

wants to merge 3 commits into from

Conversation

rubysdeadname
Copy link
Contributor

No description provided.

@TheSTL
Copy link
Member

TheSTL commented Oct 11, 2019 •
edited
Loading

Hello @gabrielbarker thanks for the PR.

  • We are not getting your commit properly . Please setup you git properly.
    image

  • And can you add time complexity for the algorithm?

@ashokdey ashokdey requested a review from TheSTL October 12, 2019 06:24
@rubysdeadname
Copy link
Contributor Author

rubysdeadname commented Oct 12, 2019 •
edited
Loading

Hi @TheSTL @ashokdey , sorry about that! I have added time complexity and will make a new PR to see if that solves the problem.

@TheSTL
Copy link
Member

TheSTL commented Oct 12, 2019

I'm closing this PR.
Fixed PR can be found here [#78 ]

@TheSTL TheSTL closed this Oct 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TheSTL TheSTL Awaiting requested review from TheSTL

Assignees

@rubysdeadname rubysdeadname

Labels
under-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rubysdeadname @TheSTL @ashokdey