Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
None
Description
Often times we add multiple fixup commits to a pull request based on review comments, but forget to squash before merging. This leads to a slightly messier commit history where a single commit doesn't show the full story of what chnaged.
To avoid this, we should add a custom GithubActions check that determines the number of commits in a pull request and either shows a check failure, or disalllows merging completely.