I believe we should be doing a rebase rather than pull and making a merge commit.
As far as 1 commit per pull request is concerned we should allow multiple commits as for the reviewer it becomes easy to review only the latest commit rather than re-reviewing the older commits. On Thu, Dec 29, 2016 at 4:53 PM, Pracheer Agarwal <pracheeragar...@gmail.com > wrote: > +1 > > On Thu, Dec 29, 2016 at 4:53 PM, Sandeep Samudrala <sandys...@gmail.com> > wrote: > > > Makes sense. > > > > On Thu, Dec 29, 2016 at 3:55 PM, Pallavi Rao <pallavi....@inmobi.com> > > wrote: > > > > > While creating a PR, we tend to push the local commits as is when we > do a > > > git push into the forked repo. These local commits include lot of > merge > > > commits, and intermediate commits. This makes it hard to review, when > > some > > > changes have been requested (as the reviewer has to either look at > > changes > > > made to each commit, or re-review the whole patch). > > > > > > Would like to request contributors to squash the new commits into one > > > before they do a git push, so, all your intermediate changes go in as > one > > > commit. Basically, resulting 1 commit per update to PR. > > > > > > > > > Thoughts? > > > > > > -- > > > _____________________________________________________________ > > > The information contained in this communication is intended solely for > > the > > > use of the individual or entity to whom it is addressed and others > > > authorized to receive it. It may contain confidential or legally > > privileged > > > information. If you are not the intended recipient you are hereby > > notified > > > that any disclosure, copying, distribution or taking any action in > > reliance > > > on the contents of this information is strictly prohibited and may be > > > unlawful. If you have received this communication in error, please > notify > > > us immediately by responding to this email and then delete it from your > > > system. The firm is neither liable for the proper and complete > > transmission > > > of the information contained in this communication nor for any delay in > > its > > > receipt. > > > > > > -- _____________________________________________________________ The information contained in this communication is intended solely for the use of the individual or entity to whom it is addressed and others authorized to receive it. It may contain confidential or legally privileged information. If you are not the intended recipient you are hereby notified that any disclosure, copying, distribution or taking any action in reliance on the contents of this information is strictly prohibited and may be unlawful. If you have received this communication in error, please notify us immediately by responding to this email and then delete it from your system. The firm is neither liable for the proper and complete transmission of the information contained in this communication nor for any delay in its receipt.
