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.

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

Added Ternary_Search.php #2375

Closed
wants to merge 4 commits into from

Conversation

BhardwajShweta25
Copy link
Contributor

@BhardwajShweta25 BhardwajShweta25 commented Mar 9, 2020

Fixes #2176

Checklist:

  • 4 space indentation.
  • Coding conventions are followed.
  • Input is taken dynamically.
  • Sample Input / Output is added at the end of file.
  • Logic Documentation (Comments).
  • File names are correct.

Changes proposed in this pull request:

  • Added Ternary_Search.php

Languages Used:

  • PHP

Files Added:

  • Ternary_Search.php

SakshiShreya
SakshiShreya previously approved these changes Mar 19, 2020
Copy link
Collaborator

@SakshiShreya SakshiShreya left a comment

Choose a reason for hiding this comment

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

Good job.

@SakshiShreya
Copy link
Collaborator

Please squash your commits.

@BhardwajShweta25
Copy link
Contributor Author

Please squash your commits.

Done, Please Check.
Thank You!

@BhardwajShweta25
Copy link
Contributor Author

Could you please check the changes?

@BhardwajShweta25
Copy link
Contributor Author

@singh-shreya6 Please see to it.
Thank you

Copy link
Collaborator

@singh-shreya6 singh-shreya6 left a comment

Choose a reason for hiding this comment

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

Please squash your commits @BhardwajShweta25. Please do it before tomorrow.

@BhardwajShweta25
Copy link
Contributor Author

I was facing trouble squashing the commits. Hence, just opened a new Pull Request #3025
Will this do?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ternary Search
3 participants