Jump to content

Code review

Code review

I checked your patches the other day and I see you are doing well. Code review is a bit slower these days because Siebrand is less active, but I'll make sure all your patches are eventually merged. Can I "assign" your reports to you, when you have already identified the solution in the report? Then you can assign them to me (by editing the support template in the thread summary) when you're unable to handle them, but I'll know which requests I need to get the developers' attention for.

Nemo (talk)15:12, 26 April 2015

Yes.

Purodha Blissenbach (talk)23:01, 26 April 2015

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
 
 
 
 
 

I've cleared the backlog of requests to me. Siebrand is also active reviewing patches here at the hackathon, so you may want to send some for a fast review. :)

Nemo (talk)05:42, 24 May 2015