Done 🙂

I would even remove the what. Because the change set already shows you what changed.

Most of the time, I try and avoid it as much as possible. There is little to no need to create abstraction in the first place when implementing something new.

Adding abstraction too early in development adds unnecessary complexity. Better to just stay simple.

Funny thing: If you use mutation testing, these magic numbers will get mutated and most likely break your tests.

So using mutation testing it actually gives you a little "hint" to use a constant value 😉

Got mine also last Friday. Sadly I didn't have any time to get some reading in. Tomorrow while commuting, there is enough time there 😅