Rain
Looks great -- yeah, just missing new tests otherwise this is good to go.
1. Good question. I think it should be marked as passed. 2. Uhmmm, there aren't good tests for deserializing JUnit reports. But there probably should be. I wouldn't worry about...
> Uhmmm, there aren't good tests for deserializing JUnit reports. But there probably should be. I wouldn't worry about that, at some point I'll have Claude Code or something write...
@eduardorittner I think not retrying is correct -- I also don't think a warning is necessary here, because the rules are very simple (pass = no retry, fail for any...
I'm probably missing something but it looks like some of the comments you marked resolved aren't actually resolved yet -- are you planning to work on them?
Catching up with things after a week off -- have a big backlog, planning to look at this later this week.
With apologies, I need to tackle something urgently at work this week. Hoping to look at this by the end of the week.
Can you just check for any SCCs with 2 or more nodes? That's how I've done this in the past.
Ah I think we should just reject that case separately, no good reason to even try to model it or support it.