And you guessed it, they copied that ignored-annotation, too, complete with the comment “//TODO currently broken”. The test they implemented was not run, not even once.

Junior dev here, this is the kind of stuff that is supposed to prompt you to ask questions. Literally one slack message when you see it, and give it a few hours for whenever you have time to see it and respond. I know it’s annoying messaging y’all with stuff but… C’mon you gotta do it sometimes, especially if it’s something you can reasonably predict will come up in review or is a simple question that doesn’t require a lot of time to answer.

Correction: modern C++ is pretty sweet.

Chances are if you’re working on C++ in a professional setting it’s not modern C++ :(

Borrow checker intensifies