-
Notifications
You must be signed in to change notification settings - Fork 43
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
Add mission timeouts #113
Comments
I can include this in my next PR |
Perfect |
@mattlangford Did we ever land on this? |
No reply from @forrestv yet. What do? |
Have you tried emailing god-president Trump? |
I couldn't find the timeout code on your fork, did you remove it? I was going to clone and try |
I did remove it. I can add it back tonight. |
Huh? Not sure what you mean. Are you waiting on me fixing the spurious traceback that is printed in the case you emailed me? I thought it wasn't blocking any progress..? |
Hmm I guess you're right. It doesn't actually break anything, no reason to fret over it. |
Well, I'm definitely going to look into it. I just wanted to point out that I'm still a bit confused, though: that issue you showed me, I wonder why I FYI, also, I'm likely going to redo the txros action API. It's a mess right On Wed, May 18, 2016 at 9:47 AM, mattlangford notifications@github.com
|
@jpanikulam, have you tried running it yet? I wonder if it's just a weird issue on my machine. I haven't ran it anywhere else. Happy birthday by the way Forrest |
Will be done via #320 |
No description provided.
The text was updated successfully, but these errors were encountered: