Skip to content

Solution of Leetcode Problem: 98 #886

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
Merged
merged 3 commits into from
Jun 10, 2024
Merged

Solution of Leetcode Problem: 98 #886

merged 3 commits into from
Jun 10, 2024

Conversation

Anshika14528
Copy link
Contributor

Related Issue

closes: #872

Description

Added solution to leetcode problem 98 (Validate Binary Search Tree)

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Checklist

  • I have performed a self-review of my code.
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Resources for Guidance

Here are some resources that may be helpful as you contribute to CodeHarborHub:

Resources for DSA Features on Our Site:

@deepsource-io deepsource.io
Copy link
Contributor

deepsource-io bot commented Jun 9, 2024 •
edited
Loading

Here's the code health analysis summary for commits 5a3d93e..bace8d5. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@ajay-dhangar ajay-dhangar added documentation Improvements or additions to documentation GSSOC'24 GirlScript Summer of Code | Contributor level2 GirlScript Summer of Code | Contributor's Levels gssoc GirlScript Summer of Code | Contributor labels Jun 9, 2024
ajay-dhangar
ajay-dhangar requested changes Jun 9, 2024
View reviewed changes
Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

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

update your code based on suggestions

@Anshika14528
Copy link
Contributor Author

Done
please review it now @ajay-dhangar sir

@Anshika14528
Copy link
Contributor Author

@ajay-dhangar sir , i have corrected the changes requested please review it now

ajay-dhangar
ajay-dhangar requested changes Jun 10, 2024
View reviewed changes
Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

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

update based on suggestions

@Anshika14528
Copy link
Contributor Author

@ajay-dhangar sir, is it ok now i have made the chnages

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ajay-dhangar ajay-dhangar added level0 GirlScript Summer of Code | Contributor's Levels and removed level2 GirlScript Summer of Code | Contributor's Levels labels Jun 10, 2024
ajay-dhangar
ajay-dhangar approved these changes Jun 10, 2024
View reviewed changes
Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

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

Done

@ajay-dhangar ajay-dhangar added the duplicate This issue or pull request already exists label Jun 10, 2024
@ajay-dhangar ajay-dhangar merged commit 1dca521 into codeharborhub:main Jun 10, 2024
5 checks passed
@Anshika14528
Copy link
Contributor Author

@ajay-dhangar sir , why is it duplicate ?

@Anshika14528
Copy link
Contributor Author

@ajay-dhangar sir , It is not duplicate it's just because of some mistake my all commits have been added in this pull request "Added leetcode problem:226 #913"
if you cannot give level to this please give level3 to "Added leetcode problem:226 #913" Pleaseee

@Anshika14528
Copy link
Contributor Author

My all three issues solution(issue no. #872 , #883) was there in this pull request "Added leetcode problem:226 #913"

@Anshika14528 Anshika14528 mentioned this pull request Jul 28, 2024
Merged
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajay-dhangar ajay-dhangar ajay-dhangar approved these changes

Assignees

@Anshika14528 Anshika14528

Labels
documentation Improvements or additions to documentation duplicate This issue or pull request already exists gssoc GirlScript Summer of Code | Contributor GSSOC'24 GirlScript Summer of Code | Contributor level0 GirlScript Summer of Code | Contributor's Levels
Projects
@CodeHarborHub
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Leetcode Problem: 98
2 participants
@Anshika14528 @ajay-dhangar