Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Added instance methods to determine the relationship between 2 nodes #314

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

Merged
mceachen merged 2 commits into ClosureTree:master from r4do:master
Jun 9, 2018

Conversation

@r4do
Copy link
Contributor

@r4do r4do commented Jun 6, 2018

Added methods parent_of?, root_of?, ancestor_of?, descendant_of?, child_of? for checking relationship between nodes

mceachen, n-rodriguez, and rnitta reacted with thumbs up emoji
Copy link
Collaborator

mceachen commented Jun 6, 2018

Thanks for the PR! Please add tests.

(In this case the diff is very straightforward, and I'm not worried about correctness, but tests prevent subsequent changes from breaking this functionality in the future).

Copy link
Contributor Author

r4do commented Jun 8, 2018

@mceachen I've added tests, please check.

mceachen reacted with hooray emoji

@mceachen mceachen merged commit 52531f5 into ClosureTree:master Jun 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /