Skip to content

/ algos

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

Jump to bottom

added Lowest common ancestor[Java] #738

Open
wants to merge 1 commit into
base: master
from
Open

added Lowest common ancestor[Java] #738

wants to merge 1 commit into from

Conversation

@sam20code
Copy link

@sam20code sam20code commented Aug 30, 2020 •
edited

Fixes #674

By submitting this pull request I confirm I've read and complied with the below declarations.

  • I have read the Contribution guidelines and I am confident that my PR reflects them.
  • I have followed the coding guidelines for this project.
  • My code follows the skeleton code structure.
  • This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
  • This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.
  • This pull request shall only be reviewed and merged once the Travis build passes. No maintainer or supporter shall be obliged to review it before this condition is met.
  • I have mentioned the issue number correctly (with hyperlink) in this pull request description.
@sam20code
task complete
Loading status checks…
2b2444f
@theyashshahs
Copy link
Member

@theyashshahs theyashshahs commented Aug 31, 2020

@sam20code mention the FIXED ISSUE and change the PR title to the work you have pushed
@sam20code sam20code changed the title assigned task complete added Lowest common ancestor[Java] Aug 31, 2020
@sam20code
Copy link
Author

@sam20code sam20code commented Aug 31, 2020 •
edited

@yashshah2820 now is it okay sir
@AmanRaj1608
AmanRaj1608 requested changes Aug 31, 2020
View changes
Copy link
Member

@AmanRaj1608 AmanRaj1608 left a comment

Please indent code as per editorconfig
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AmanRaj1608 AmanRaj1608

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

Lowest Common Ancestor [Java]
3 participants
@sam20code @theyashshahs @AmanRaj1608
You can’t perform that action at this time.