News for hackers. Stuff that matters.

A comment on Stacked changes: how FB and Google engineers stay unblocked and ship faster.

Re:Stacked changes: how FB and Google engineers stay unblock... by coderdd on Thursday November 18, 2021 @02:56AM (#29263154)
I worked at Google and have no idea what they mean by stacked change. You could push multiple changes to a single CL, but those were rarely reviewed independently.

Maybe they mean keeping a stack of CLs ready, while waiting for a review? I often had 1-2 CLs based on.. rebasing was fun sometimes.

Or they mean diffbasing these subsequent CLs, for an early review? That was useful, but due to volatility dependent on the first review, a bit risky.

Plug for a code review project: https://reviewpad.com/

[ Parent | on HN ]
Re:Stacked changes: how FB and Google engineers stay unblock... by exclipy on Friday November 19, 2021 @06:10PM (#29283774)
Stacked diffs is Facebook terminology for diffbased CLs (Google). Except Facebook has invested a lot of tooling into it to make it possible without many of the downsides (see the comments here about 10+ diffs stacked on top of each other).
[ Parent | on HN ]