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.
Whenever you have applications were implementations are plentiful the only real differentiation you can do without creating a different user experience is the technologies used to develop it. The importance of which in people’s perspective is several things, mostly supporting technologies they like and want to see grow and possibly being skilled in the underlying technologies to actually contribute back.
Certain technologies are also just hot garbage, I swear to God if I have to install another electron app for some messaging platform I will shit myself.