from me to rebasing in pr tests, that would also allow us to run e.g. [test foreman] again before merging instead of asking contributors to rebase every time a pr gets a little stale.
Looks like the current behaviour was introduced 3 years ago due to some issue with the commit sha being changed when rebasing:
Perhaps someone from @infra team knows if this is still relevant? Apparently Dominic didn’t eventually get around to restoring the previous behaviour.
from me to rebasing in pr tests, that would also allow us to run e.g. [test foreman] again before merging instead of asking contributors to rebase every time a pr gets a little stale.