Skip to content

Added mode option for the unroot function#344

Merged
jhcepas merged 2 commits intoetetoolkit:masterfrom
gguta:master
Apr 6, 2018
Merged

Added mode option for the unroot function#344
jhcepas merged 2 commits intoetetoolkit:masterfrom
gguta:master

Conversation

@gguta
Copy link

@gguta gguta commented Apr 1, 2018

This is a simple solution for the issue #269 It maintains compatibility with the previous versions.

@jhcepas
Copy link
Member

jhcepas commented Apr 2, 2018

thanks @gguta ! this is useful.

could you add a couple of test lines too?

t = Tree('(A,B,(C,D)E)root;', format=1);

@gguta
Copy link
Author

gguta commented Apr 2, 2018

Hi @jhcepas ,
I have added the requested test with "test_unroot(self)" signature. I hope this is sufficient.

@gguta
Copy link
Author

gguta commented Apr 5, 2018

Hi @jhcepas , I have added tests as you requested 3 days ago. Do you need any further contribution from my side to accept this pull request?

@gguta gguta closed this Apr 5, 2018
@gguta gguta reopened this Apr 5, 2018
@jhcepas jhcepas merged commit da6a23a into etetoolkit:master Apr 6, 2018
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