Andy Soffer

Results 18 comments of Andy Soffer

Thanks. I don't see an immediate problem with this but I don't think we're going to find time to prioritize this effort. I'd suggest using the json or xml output...

Apologies for the delay. This looks resaonable. I'll pull it in.

Again, sorry for the delay. I've been having trouble pulling this in. could you sync this to head and I'll try again?

Can you say a bit more? What do you mean by "processing"? By "streaming" do you mean text written to `stderr` and `stdout`?

@antasp please address the failures in the CI. If you could also asnwer the question I left on the code, I would really appreciate it. Thanks!

@antasp, so sorry. I forgot to submit the review.

Thanks for the explanation and sorry for the delayed response. Given that this is already disabled globally for the project, we would prefer not to add additional handling related to...

Generally looks good. Please sign the CLA, and we'll get this merged.

I personally find myself advocating on the other side: anything remotely nonsensical (integer overflow... regardless of signedness, division by zero, null dereference, etc) should all be undefined, because it *helps...

Thanks! We discussed this and generally like the idea. Due to the nature of the change this is a thing we'd like to design and take through some internal API...