If you min(1, 2, null), you get null.

I have reviewed this exhaustively and can confirm that it's a big file full of SQL 👀

we can cache datasets by chart key very near the display layer

That screenshot produces quite the intense optical effect. I thought I was looking at moiré effect from a cellphone camera.

A half-step might be to just silence "added a commit to" in all cases?

All the same underlying action:

I feel kinda similarly about getting a flurry of "added commit to TXXX" notifications that fire in addition to the "bob closed DXXX" notifications. We don't have the infrastructure to say stuff like "only hide this notification if the same action already generated a separate notification of type foo, do we?

I guess it's not worth cleaning this up and removing the class + DB rows since it's just a no-op now and shouldn't bother anyone.