100% this. Makes it so much clearer to see what you are reviewing, when the commits are logical.
The number of times there are seemingly random changes either bundled into a random commit (which makes it really hard to get context over why the change was made), or as you call out "Fixing PR comments".
I think developers who have experienced the fall out of this, agree with it. The developers who haven't had to deal with issues in git or use it to solve an issue, don't see the value.
So, as always, it's experience, in my opinion.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
100% this. Makes it so much clearer to see what you are reviewing, when the commits are logical.
The number of times there are seemingly random changes either bundled into a random commit (which makes it really hard to get context over why the change was made), or as you call out "Fixing PR comments".
I think developers who have experienced the fall out of this, agree with it. The developers who haven't had to deal with issues in git or use it to solve an issue, don't see the value.
So, as always, it's experience, in my opinion.