Damus
aeva · 2w
[approves 10,000 line pull request, slowly removes glasses and thoughtfully cleans them with a small cloth] I honestly don't think any bugs are in there.
Lou. profile picture
@nprofile1q... remind that one time one dev brought us a PR that was like 14k lines, was around 4 month of work including big change that had no business being done in that branch. I think they all expected me to review it as I was kinda the other backend dev. I was very clear, I don't review that amount of code. If they want to merge it without anyone approval, they go ahead.
Of course that was merged and I kept hitting that code, it was hell and I couldn't really complaint as I acceptes the situation