it is beneficial to preserve nicely organized logical commits in pull requests
Description
Although it makes sense for pull requests to be squashed when they are made of multiple incremental commits added each time a reviewer asked for a change, it should not be a requirement.
In the past it was difficult for the reviewer to visualize the changes after the author of the pull request force pushed a commit series with just a typo change. But it can now conveniently be seen when clicking on the *force-pushed* link added in the pull request comments. It only displays the diff between the old commit and the new one that was force pushed.
Steps to Reproduce
None
Current Behavior
None
Expected Behavior
None
Reason for Variance
None
Release Notes
None
User Impact Summary
None
Activity
Show:
singuliere singuliere
February 22, 2019, 10:14 AM
Proposed an update at https://github.com/edx/edx-documentation/pull/1803
Assignee
Reporter
Labels
None
Reach
None
Impact
None
Platform Area
None
Customer
None
Partner Manager
URL
Contributor Name
None
Groups with Read-Only Access
None
Actual Points
None
Category of Work
None
Platform Map Area (Levels 1 & 2)
None
Platform Map Area (Levels 3 & 4)
None
Priority
