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 _______________________________________________ Pywikipedia-l mailing list Pywikipedia-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l
