Skip to content

Conversation

@AhmedHamdiy
Copy link
Contributor

In this Pull request,

  • I have fix addChild: is not in sync with its implementation.
  • I added parent attribute to the AVLNode and tested it.

@AhmedHamdiy AhmedHamdiy changed the title fix: addChild: is not in sync with its implementation Fixes #29 Mar 27, 2025
@AhmedHamdiy AhmedHamdiy reopened this Mar 27, 2025
@AhmedHamdiy
Copy link
Contributor Author

@Ducasse Could you review this pr please

@jordanmontt
Copy link
Member

Could you explain the changes in this PR?

@AhmedHamdiy
Copy link
Contributor Author

The most significant changes include adding parent references to nodes, updating node addition methods to maintain parent relationships, and adding methods to retrieve all children of a node recursively.

Parent-Child Relationship Management:

Node Addition:

Node Access Methods:

Tests:

@jordanmontt jordanmontt merged commit 339dd32 into pharo-containers:main Apr 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants