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.
