Answer the question
In order to leave comments, you need to log in
What's wrong with the number of commits a little more than for which the task could actually be solved?
I got a new job, set a task, made one commit in the branch, they said at the review that I could solve it a little differently. As a result, I made 4 more commits, the reviewer says: "there are already 5 commits, let's get a new PR".
What is the problem with 5 commits, can someone tell me?
Answer the question
In order to leave comments, you need to log in
In the personal cockroaches of the reviewer. If the commits soar when merging the PR, you can mold the commits into one.
On the contrary, the fewer changes in the commit. all the better. I suffer from the opposite, at the end of the day you make a commit, then you won’t understand all this.
in an ideal world where everyone poops butterflies, a commit should be a complete phase of some kind of editing, implying that a specific (sub)task has been completed and a fix has been made to the system that does not require modifications and guarantees the system to work.
In real life things are different.
your 5 commits - from excitement.
in fact, you just had to check everything and make another commit.
why is there PR I don’t understand
Didn't find what you were looking for?
Ask your questionAsk a Question
731 491 924 answers to any question