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

feat: add LeetCode problem 69 #1259

Merged
merged 7 commits into from
Jun 27, 2023

Conversation

Asp-Codes
Copy link
Contributor

@Asp-Codes Asp-Codes commented May 9, 2023

Here is the code for the problem 69 of leetcode as there are many ways to do it we programmers need to find the most optimal way to solve a problem statement so i used binary-search approach inorder to solve it. All suggestions are accepted!!!

Description of Change

References

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:

Here is the code for the problem 69 of leetcode as there are many ways to do it we programmers need to find the most optimal way to solve a problem statement so i used binary-search approach inorder to solve it.
All  suggestions are accepted!!!
@github-actions github-actions bot added the Leetcode folder changes Changes to Leetcode folder. Known CI issues. label May 9, 2023
@alexpantyukhin
Copy link
Member

Why you closed the previos MR? you could make changes in the context of the prev.

@CascadingCascade
Copy link
Collaborator

Why you closed the previos MR? you could make changes in the context of the prev.

Check out #1257 (which is closed by me for being a duplicate), they refuse to do that for whatever reason

leetcode/src/69.c Outdated Show resolved Hide resolved
leetcode/src/69.c Outdated Show resolved Hide resolved
I have updated the solution according to the suggestions.
leetcode/src/69.c Outdated Show resolved Hide resolved
alexpantyukhin
alexpantyukhin previously approved these changes May 22, 2023
leetcode/src/69.c Outdated Show resolved Hide resolved
@github-actions
Copy link
Contributor

This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale label Jun 22, 2023
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Thanks! 🚀

@Panquesito7 Panquesito7 merged commit db69d0a into TheAlgorithms:master Jun 27, 2023
@Asp-Codes Asp-Codes deleted the solution/69.c-2 branch June 28, 2023 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Leetcode folder changes Changes to Leetcode folder. Known CI issues.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants