Among the options merge commit , rebase and squash merge, rebasing is the problematic one if the commits on the branch are not carefully crafted. Classical merging allows for bisecting and retains historicy.
But IMHO if swash merged lead to a situation that relevant history is lost the PR is too big.
But IMHO if swash merged lead to a situation that relevant history is lost the PR is too big.