Comment by psd1

I've come out of a job of spaghetti and idiots.

My team lead used to shoot down ideas if they were "too complex". Correct and wise, right?

Except that this numskull created reams of functions that took untyped python dictionaries, did something, and passed them to other functions.

Early on, he bolted a giant mess of validation onto the perimeter. This did add value... but most data invalidity arose inside the fucking app!

He had a great eye for the complexity of abstractions, but was completely blind to the complexity of doing simple things in convoluted ways.

It was miserable. Any random PR could pass all the integration tests we had and still blow up prod, because there were orders of magnitude more code paths than lines of code.