Jump to content

Code review

Ok, I'm proceeding. I'm fixing most of the things myself, but those assigned to you are still multiple. Most of them don't take much time and I left some hints. The most tedious ones are PLURAL and GENDER support, I suggest to file reports in phabricator when you see that a patch would take too long.

Nemo (talk)10:33, 14 May 2015
  1. Reporting in phabricator when I see that a patch would take too long or was too complicated is exactly what I would like to do.
    But remember: phabricator is dead for me.
  2. PLURAL and GENDER support were among the easiest things I did for long. But they require imho that I test them (unlike pure message text amendments) and I do not have access to a working MediaWiki test installation. The latest failure to get one was that tools lab does not have composer, if I recall that right, and I cannot request that changed, see above.
    Thus I unset my name from all such tasks assigned to me at the moment.
Purodha Blissenbach (talk)15:58, 14 May 2015
  1. I work under the assumption that WMF ops will eventually fix Phabricator for you... You could try emailing ops-requests@rt.wikimedia.org to ask help, IIRC this bridge still works.
  2. In many cases, GENDER and PLURAL don't require testing: sometimes they don't even need code changes, because the developer just forgot to use the magic word in the English text. Anyway, that's fine; please assign the PLURAL and GENDER stuff back to me when you can't handle them, so that I remember to do what needed.
Nemo (talk)07:20, 15 May 2015
  1. Just sent an e-mail.
  2. Yes, you are right. It is only frustrating when I find that I have a fix but cannot test and submit it. Maybe I should learn to live with the risk ;-)
Purodha Blissenbach (talk)07:42, 15 May 2015
  1. Hm, not sure it worked, we'll see.
  2. If you think more testing is needed, you can just -1 yourself and say so. Especially for WMF-developed extensions, filing a patch is just a nicer (and often easier) bug report coming with a clear solution proposal. It's still their responsibility to fix; people with +2 rights can and should test where needed.
Nemo (talk)08:05, 15 May 2015