I'm two days into
peer review and I'm three stories behind. There's this guy, let's name him Travis. He says his code is perfectly fine, but
it doesn't really sit with me quite right. Because it seems like
there's extra things in his lines, and I keep accidentally pushing bad code into production
in the middle of the night. So I'm about to push, and I'm up late
at night thinking the code looks right. When I see hard-coded
sys_ids, bad in practice, worse on paper. Couldn't you have used a property
or even a config table? And for extra brownie
points, just stop hardcoding things.