Skip to content

Conversation

@idoadiv
Copy link

@idoadiv idoadiv commented Sep 12, 2021

No description provided.

@idoadiv idoadiv closed this Sep 12, 2021
@guillermo-navas-palencia
Copy link
Owner

The PR looked good. Any reason for closing it?

@idoadiv
Copy link
Author

idoadiv commented Sep 13, 2021

It was closed by mistake

@idoadiv
Copy link
Author

idoadiv commented Sep 19, 2021

Are you waiting for something in order to merge this?

@guillermo-navas-palencia
Copy link
Owner

Yes. Force termination is not properly documented, and should not be passed as a dict key of data. In addition, this behaviour could be obtained outside the class. Therefore, this needs to be changed before merging. The amendments for darwin are both alright.

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.

5 participants