Skip to content
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

Capture errors from threads #41

Closed
wants to merge 1 commit into from
Closed

Capture errors from threads #41

wants to merge 1 commit into from

Conversation

Jc2k
Copy link
Collaborator

@Jc2k Jc2k commented Dec 21, 2018

To help debug #40.

This is untested - I don't have an ESX instance to hand - but hopefully will help until i'm back in the office.

@dannyk81
Copy link
Collaborator

@Jc2k, I think this is not needed anymore, since it was added in #43, right?

@Jc2k
Copy link
Collaborator Author

Jc2k commented Dec 23, 2018

Yep, and then it’s removed again in the other PR as twisted does it a different way 😅. Either way can close this.

@Jc2k Jc2k closed this Dec 23, 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