Sometimes I like to look at my PR and think "Damn, that's some beautiful code, I can't wait for someone to review it".

Then the review comes and see a bunch of requested changes. =/

@dUcKtYpEd What i hate the most is the project manager that wants things to be merged quickly without reviewing and testing fully.

@dUcKtYpEd That is how it should be. Here at my side : Every pull/merge request is checked by only a Junior developer and that Junior has to test the app briefly to not let any defects go in. That junior also codes and merges his own requests...

@dUcKtYpEd Project lead is busy with backend stuff, since the other company can't provide services properly. So that junior leads the project...

@dUcKtYpEd So is it a good opportunity to improve for the junior ? Interesting.

I haven't thought of it like that. I thought he has more responsibility on his shoulder than he could carry. But when you put it like that, it makes a lot sense

@dUcKtYpEd I get you, not everything is programming. We need to learn management and leading too :)