Hi, We do not squash commits unless they are related (Which should have been done by the OP in the firsr place). We actually encourage putting unrelated changes in separate commits.
On Thu, Mar 5, 2020, 05:26 Takashi Yamamoto <yamam...@midokura.com.invalid> wrote: > hi, > > it seems that in nuttx it's common to squash commits when merging pull > requests. > > i'd like to suggest _not_ to do that because: > * it makes cherry-pick/backport/revert cumbersome > * larger changes are more difficult to read/understand > > on the other hand, i can think of only little benefit. > * aesthetic reasons? > * marginally save the repo size growth? > > how do you think? >
