Skip to content

Add PatchSet.DiffStat field and use that instead of computing it client side

Warren Gifford requested to merge campaigns/patchset-diffstat into master

Created by: mrnugget

This is part 2/2 to fix #7092 (closed) and introdues the diffStat property on PatchSet.

Just like in #10170 this here includes no caching. For now I want to hold off on adding caching until I have a clear idea of how to invalidate said caches. I don't think it's a high priority right now. Will also document this in #7092 (closed).

@eseliger Can you take a closer look at the removal of the nested patches { nodes { diffStat { ... } } } in the query? I'm not sure how to find out whether that's used anywhere else or not (we do display single diffstats, but I'm nor 100% sure whether we need that query for it)

Merge request reports

Loading