Although I would say that that's a good practice nonetheless, because it's true that each commit gets a merge commit between the PR and master (so it's not the PR branch directly being tested), but that happens only if you commit, and if something lands on master that potentially breaks the PR, you won't notice