Re: [PATCH v2 0/3] {apply,alias}: convert pre-processor macros to enums
- From
Vinayak Dev <vinayakdev.sci@gmail.com>
- Date
- Feb 11, 2023, 04:00 UTC
- Message-ID
- <CADE8NaqvP1JbLL89prOCu2Qo-ZVJTTN_CZjk7bJwzMctV9Z2Bw@mail.gmail.com>
- In-Reply-To
- <xmqq8rh5gox7.fsf@gitster.g>
On Sat, 11 Feb 2023 at 03:31, Junio C Hamano <gitster@pobox.com> wrote:
Show 22 quoted lines
> > Vinayak Dev <vinayakdev.sci@gmail.com> writes: > > > Revert changes to alias.c, and change variable types in apply.c > > When you send out a rerolled series, you do not have to show your > past mistakes. This [v2] is structured as a three patch series > whose first one makes a similar mess as what [v1] did, the second > one and the third one then revert some parts of that earlier mess. > > That is not what you want to show your reviewers, and more > importantly, that is not what we want to record in our history. > Rerolling a series is your chance to pretend that you are much > better programmer than you who wrote the [v1] patch. The review > exchange is to help you do that. Please take advantage of that. > > You may find "git rebase -i" is a useful tool to help you pretend > that you got to the ideal end result without these "I tried this > first, which was wrong in these points, which I correct in a > subsequent step" steps. > > Thanks.
OK, I will keep that in mind before sending subsequent patches. Should I re-send [v2] after making corrections for this mistake? That would make the corrections more obvious and the mistakes less.
Thanks a lot! Vinayak