Hi this is probably not important for long if we transition to
Phabricator, but I want to ask if developers could rebase only if
necessary (merge conflicts with master) and if you rebase do it in a
separate patchset.
The reason being that when you compare two patchsets to determine the
changes to a version you've already reviewed that this is ugly if
there was a rebase in between as the changes due to the rebase are
included in that.
This means gerrit now show unrelated changes and it's harder to
determine which change were introduced due to the rebase and which
were intentionally added by the dev to the new patch set.
Thank you in advance ;)
For example the change to generate_user_files between PS25 and PS26
[0] are neither a part of PS25 nor PS26.
Fabian
[0]:
https://gerrit.wikimedia.org/r/#/c/195105/25..26/generate_user_files.py