Skip to content

feat: add LeetCode problem 69#1256

Closed
Asp-irin wants to merge 1 commit intoTheAlgorithms:masterfrom
Asp-irin:solution/69.c
Closed

feat: add LeetCode problem 69#1256
Asp-irin wants to merge 1 commit intoTheAlgorithms:masterfrom
Asp-irin:solution/69.c

Conversation

@Asp-irin
Copy link
Contributor

@Asp-irin Asp-irin commented May 4, 2023

Here i am with the two methods,use can use accordingly as your wish,all suggestions and critics 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 i am with the  two methods,use can use accordingly as your wish,all suggestions and critics are accepted
@github-actions github-actions bot added the Leetcode folder changes Changes to Leetcode folder. Known CI issues. label May 4, 2023
@alexpantyukhin
Copy link
Member

Please follow the https://github.com/TheAlgorithms/C/tree/master/leetcode#-adding-a-new-solution- doc. Actually you have to put the solution which could be run on the leetcode. Your solution will fall because of it contain 2 declarations of the int mySqrt(int x){.

@Asp-irin Asp-irin closed this May 9, 2023
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.

2 participants