Replies: 1 comment
-
As of an example on this PR #301 Is it OK to use I like to commit each time the test passes and more. Ideally |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
It occurred to me in one of the PRs that it's hard to make sense of a PR when you're close to a merge if we're always doing squash. My suggestion would be to never squash a PR by the owner, only squash it on merge. That way it might remain more understandable what the evolution of the development was, what and why a particular commit changes in the PR.
Certain things can be squashed, but the logic of the evolution should be traceable.
Beta Was this translation helpful? Give feedback.
All reactions