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.

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

Depth First Search #794

Merged
merged 4 commits into from
Oct 22, 2020
Merged

Depth First Search #794

merged 4 commits into from
Oct 22, 2020

Conversation

adityak3252
Copy link
Contributor

@adityak3252 adityak3252 commented Oct 22, 2020 •
edited
Loading

Have you read the Contribution Guidelines?

(yes)

Description

(Depth Search Algorithm is added in C-plus-plus folder and Readme is also updated.)

Checklist

  • [ ✔️] I've read the contribution guidelines.
  • [✔️ ]1 I've checked the issue list before deciding what to submit.
  • [ ✔️] I've edited the README.md and link to my code.

Related Issues (Mention Issue no. like #xxx)

#792

Fixes Issue
Issue is fixed.

@adityak3252 adityak3252 mentioned this pull request Oct 22, 2020
Open
@adityak3252 adityak3252 changed the title Dfs branch Depth First Search Oct 22, 2020
@sunilgknair051
Copy link
Contributor

@adityakarante , Please comment code for better readability.
Thanks!

adityak3252
adityak3252 commented Oct 22, 2020
View reviewed changes
Copy link
Contributor Author

@adityak3252 adityak3252 left a comment

Choose a reason for hiding this comment

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

please check the updated file

itslinotlie
itslinotlie approved these changes Oct 22, 2020
View reviewed changes
Copy link

@itslinotlie itslinotlie 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 on the updated code comments. Looks good enough to be merged.

@itslinotlie itslinotlie merged commit 9fba1dd into sukritishah15:master Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itslinotlie itslinotlie itslinotlie approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adityak3252 @sunilgknair051 @itslinotlie