I wanted to find out how folks feel about squashing commits for reviews on pull request. This is not for the initial pull request but only for subsequent commits to address the reviews. Here are some options.
1. Squash everything to a single commit. This is the process we are following today. Advantage is there is one commit per JIRA and self contained. Easy to cherry-pick if needed. 2. Preserve the individual commits for pull request reviews. Advantage is you preserve the review history in the pull request, the thoughts and discussions that went behind the changes and you see the incremental changes in separate commits. Disadvantage is you will have to work with multiple commits if you are trying to something with the change like re-apply it elsewhere. 3. A hybrid of 1. and 2. where you don't let the commits grow large. No standard set limit but the contributor and committer work keep it to a reasonable amount squashing smaller commits. 4. Anything you would like to propose. I would like to add +1 for option 3. Thanks
