(In reply to Cameron Dawson [:camd] from comment #4)
This is the performance data tab in the Task Details, right? As opposed to having a Perf tab be peer of Linting | Builds | Tests. I just wanted to be clear.
Yeah, exactly - just adding it as another tab along with artifacts and logviewer, the way it is in TH.
Yeah, that's a good idea. Do you have a feeling for if you'd rather have it be disabled or hidden if there is no perf data? Armen added the new
Test Groups tab to the TH details panel, but has it disabled when not applicable. I was wondering if we should do that for the Perf tab as well (it's currently hidden).
I don't have a strong opinion on that. If it's disabled, perhaps adding a tooltip on hover stating "no performance tests were ran" or "no data to show" so people don't think something's broken. Not sure if that's how Armen has done it for Test Groups.