This morning I found that Brad approved my PR, Friday, even though it was WIP, and one test was failing (in mock).
Well, I fixed the mock for the test, invited more people to review, reminded them - but nobody cared.
Later today Brad just merged my branch into his, and, seemingly, merged it to master (I already approved his code, although suggested to rename one weirdly-named method. I merged back master to my branch, but the same test started failing. So I wrote to him about that.
Esteban, who is expected to be in a hospital, wrote that he had worked on the doc, and here's the new update. The update contains a flow diagram, which looks neat, but makes no sense (a service, with no prompt, suddenly calls another service, and does not care about the result... etc. Creative UML.)
Anyway, it's unreadable, and I'll prefer to stay away from that shit. If I worked in that company... but I don't.
Spent the rest of the time investigating the weird behavior of `run()` in one service, where an exception is thrown all the way to the top, while everything is supposed to be a sequence (Stream) of monadic effects. Why Stream? No reason. It's just a sequence of operations, that's it. Instead of writing the code line by line, or Kleisli-composing them, we proudly build a Stream, and then "execute" it. During the execution, an exception is thrown, and neither streams nor IO inside sees it. Functional fucking programmers, eh.
It took me half a day (and some time before today) to manage to reproduce this dumb behavior and catch it. Now that I can reproduce it, I'll find a cats-kosher way to fix it.
That's it for today.
Do I complain? No way. No reason to complain. I'm still curious about shit.