Code review comment for ~pwlars/checkbox-ng:get-test-outcome-stats

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

I'd keep resources and attachments in the big picture as they can fail (generating less jobs for example or lead to other tricky side effects). This patchset ignores/filters them and I don't think it's a good idea if we are basing automated reviews exclusively on "real" jobs.

Where should we tweak reports to add the missing resource/attachments and keep the existing junit.xml?

review: Needs Information

« Back to merge proposal