19 Mar
2021
19 Mar
'21
11:20 a.m.
I'm not a Cinder dev, but I could not help looking at this suggested change. To me at least, it's an insta-merge, but I think it might get overlooked because the Commit Message doesn't detail "What you would now see" versus "What you should see", and the cognitive load on reviewers is high enough that it gets ignored. +1 from someone who has no right to say so. -- MC On Fri, Mar 19, 2021 at 3:43 PM Sam Morrison <sorrison@gmail.com> wrote:
Hi Cinder cores,
I've had a simple review [1] waiting for several months, could I please get some feedback on this.
Thanks! Sam