On Wed, Jan 16, 2013 at 8:06 AM, Sébastien Santoro dereckson@espace-win.org wrote:
At the end, the direct commit message edit in the UI will offer an acceptable solution: corrections will be more trivial than found again my branch, amend the commit, resubmit as a new patchset. Meanwhile, we can suffer the last weeks of extra work for review spelling (as 0 or -1) purpose pending the Gerrit migration.
And really, this is just a few weeks out. I've been testing the upgrade, and I'm confident we won't see any real problems. I'm planning to do it as soon as the eqiad migration is complete next week.
And if you don't want to fix yourselves your commit, please create a list stating so on mediawiki.org, that will be a clear message for the code reviewers: "If you see a typo, would you be so kind as to fix it yourself and submit a new patchset?".
Please no. We really don't want to encourage people to be lazy because they think people will clean up after them.
Really, I think the whole thread is moot with the pending upgrade. Typos should always be fixed before merging (I think we all agree?), and the new abilities to fix these from the UI means we won't need to mark people as -1 to do so.
-Chad