Gerrit/Project ownership

__NEWSECTIONLINK__ This page queues individuals' requests to be added to the Gerrit project owner groups for specific Gerrit projects (each of which corresponds to a Git repository). A Gerrit project owner has the power to approve changes for merger into that Gerrit project's master branch, and to veto changes (see +2).

"When/how we'll add, remove people from Gerrit project owner groups" has procedural details. Sumana Harihareswara will regularly look at new requests for project owner membership and contact the existing project owners. If there is consensus from the existing project owners, then we'll approve the candidate. For each new candidate the process shouldn't take more than two weeks, and usually much less. Ownership can be revoked.

If your codebase/extension/tool isn't in Git yet, use this form to create a new Gerrit project: Gerrit/New repositories

To see the current list of Gerrit project owners for a specific Gerrit project, visit https://gerrit.wikimedia.org/r/#/admin/groups/.

= Ownership structure = Example: an extension is named foo.
 * The Gerrit group "foo" should usually be an owner of the Gerrit project "foo."
 * Sometimes, meta-groups will be included in the group. This is for people have ownership over multiple extensions, so you can add/remove members in one place.
 * Rights to the group may be inherited from other groups (Look for a "Rights Inherit From:" in the project access.)

Specific example: the project "mediawiki/extensions/DonationInterface" is owned by group "extension-DonationInterface." This group includes the meta-group "fundraising." Also members of the group "mediawiki" has ownership via "Rights Inherit From: "mediawiki/extensions access"

By keeping the naming convention ("extensions/foo" is owned by group "extension-foo"), it'll make the "automatically setup a repo" process much more scriptable when we hit that bridge.

(Note to Gerrit group creators: remember to check the "Make group visible to all registered users." checkbox and leave the group type as "Internal group" and not LDAP.)

= To make a new Project Owner =


 * Create a group
 * Give it ownership of a Project
 * Anyone in that group can now add more owners via https://gerrit.wikimedia.org/r/#/admin/projects/ (but we prefer to keep that process public via Git/Gerrit project ownership)
 * Click Groups
 * As long as you are a member of the group, you can edit the group
 * example: https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki,access

MediaWiki core
We are maintaining a "WMF" branch of mediawiki/core.git. We use submodules for deployed extensions, and can pull from master as regularly as we want for deployments. At the start of the migration to git, the project owners of this branch are going to be the people who have the ability to deploy code to Wikimedia Foundation servers. gerrit will offer a list of the "Gerrit project owners" for this branch, except for the Operations (system administration) group, which is an LDAP group. Every member of the Wikimedia Foundation operations team will also be in the Gerrit project owners group insofar as they have code review rights globally, but in practice will rarely review code. We may add some existing code reviewers to this Gerrit project owners group. Details; you can request to be added.

At the start of the migration, this list of Gerrit project owners for the WMF branch is also the list of Gerrit project owners for the master branch. However, eventually, we will add to the list of Gerrit project owners for master, using as criteria the number and quality of developers' previous commits and code reviews.

Details and procedure for adding and removing people from the Gerrit project owners groups.

MediaWiki has release branches (19 so far) for core, and master (the default branch previously known as "trunk" in SVN). Example: ("heads" is gitweb's term for branches). MediaWiki core and WMF-deployed extensions will be tagging releases just as we did in Subversion, except they'll be Git tags instead of SVN tags. Any other extension will make its own decisions regarding tagging.

MediaWiki extensions that the Wikimedia Foundation deploys
Same procedure as for MediaWiki core, and the same Gerrit project owner groups.

Other MediaWiki extensions
Every extension author can choose between two choices here for non-master branches: the gated-trunk/push-for-review model, and a straight push model. For any given extension, we will honor the wishes of the person/s listed as the main author on the extension's mediawiki.org page.


 * The gated-trunk/push-for-review is the model that we are using for MediaWiki core, as mentioned above. A Gerrit project owners group (plus the above mentioned Gerrit project owners group for MediaWiki core) will be able to "+2" (approve and merge) changes to their extensions.  The extension author(s) will be able to define a Gerrit project owners group and add others to it.


 * The straight push model is similar to how we did things in Subversion; anyone can suggest a change and submit a pull request, and it will automatically be approved and merged.

Master branches must go through Gerrit and cannot be straight push. This is necessary to facilitate a number of Gerrit features, including replication, updating of the extension meta-repository, and ability of Translatewiki to provide localization updates.

We could define groups to make this easier for batches of extensions (e.g. SMW developers). Chad will offer your community a choice. Please let Chad what you would like via Git/New repositories.

Other Gerrit projects
Same procedure as for "other MediaWiki extensions" above.

= Requests =

[ Add a request]

A bunch of new groups
I'd like to have an $extension-trusted group (initially empty and with no rights assigned) for the following extensions: Validator, Maps, SemanticMaps, Push, LiveTranslate, SubPageList, Spark, IncludeWP, Survey, DidYouKnow, Gitweb (once created), DataValues (once created), Diff, SemanticWatchlist, SemanticImageInput and SemanticBundle.

These groups would be owned by their respective $extension-owner group.

This will allow me to manage my extension myself without posting a request here each time someone should get access :) --Jeroen De Dauw (talk) 19:25, 17 August 2012 (UTC)

Comments

 * Support. --siebrand (talk) 16:06, 21 August 2012 (UTC)
 * Since these are non-deployed extensions, I think this is fine. But I'd like to get input from Chad before we start introducing new group structures. --Catrope (talk) 18:02, 21 August 2012 (UTC)
 * I don't *like* it as proposed since this is very quickly going to explode the number of groups we have. Ideally all extension-$name groups should be owned by an extension-$name-owner (other than deployed exts, perhaps). The reason it's all currently owned by the "Project & Group Creators" was so people could process this page and add new users to their respective groups. What I'd like is either A) A way to manage all groups without giving out admin privs, or B) Multiple owners of groups. The former is probably easier. This all being said, perhaps we can go ahead with this structure for a few non-deployed extensions anyway and see if we really need this page at all for granting access to those. ^demon (talk) 14:05, 22 August 2012 (UTC)
 * Well then I suppose we'd be supportive of adding Jeroen as the owner of all those groups? Also, it seems to me that there are a number of them should be in some metagroup fro SMW. Tychay (talk) 20:56, 5 October 2012 (UTC)
 * Chad, sorry for being obtuse, but when you say "perhaps" is that a go-ahead for me to go ahead and create the "extension-Validator-trusted", "extension-Maps-trusted", etc. Gerrit groups, and when necessary, the "extension-[name]" groups to own the "trusted" groups? Thanks. Sharihareswara (WMF) (talk) 15:44, 8 October 2012 (UTC)
 * Jeroen, I set up the relevant groups for Validator and I think I set them up how you'd like. Is that right? Tried to ping you in IRC, haven't gotten a response -- let me know if that's the right model and I'll do the rest of the groups. Sharihareswara (WMF) (talk) 23:52, 9 November 2012 (UTC)

Thanks for setting that group up Sumanah. You made the group owner of itself. If this is an acceptable thing to do, then perhaps there is no point in having such extra groups, as you could just make the extension-name groups own themselves, solving the issue with less work and clutter. Think chad had some reason to not do this though. --Jeroen De Dauw (talk) 20:15, 12 November 2012 (UTC)
 * Yeah, the reason was that it made it impossible for people in the "Project and Group Creators" group to manage any group member. This is being solved in the near future. ^demon[omg plz] 15:58, 21 February 2013 (UTC)

Nischyan branch owner in SemanticMaps
I tried making Nischyan owner of the nischyan branch of SemanticMaps by adding a group he's member of as owner of this branch. I want him to be able to happily merge in stuff without needing approval of someone else. Since this did not work, I tried giving him pretty much all other rights on the branch, but he says he's still not able to approve his own commits. Am I doing something wrong? https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/extensions/SemanticMaps,access
 * -- Jeroen De Dauw (talk) 15:04, 24 September 2012 (UTC)


 * Reviewing commits (Verified+1, CodeReview+2, Submit) needs to be assigned to the appropriate refs/for/* refspace, not the destination ref. Compare and . ^demon (talk) 19:23, 24 September 2012 (UTC)


 * So it should be refs/for/refs/heads/nischyan? --Jeroen De Dauw (talk) 18:26, 28 September 2012 (UTC)


 * Yes ^demon (talk) 18:21, 18 October 2012 (UTC)

This does not appear to work - can you check the settings to see if they are correct? --Jeroen De Dauw (talk) 18:53, 21 October 2012 (UTC)

Creation of a repository for my extension, PreloadContent
I am developing a new extension, Extension:PreloadContent and I want to have my code hosted on MediaWiki repos. --Milad Naseri (talk) 11:09, 3 December 2012 (UTC)
 * Can you please describe this extension in greater detail, or provide a code sample? --ori (talk) 00:40, 28 December 2012 (UTC)


 * I think you're looking for Git/New_repositories. Bawolff (talk) 15:58, 21 February 2013 (UTC)

extension-MediaWikiAuth
As the original creator of the extension, I (GreenReaper) request project ownership for extension-MediaWikiAuth. GreenReaper (talk) 04:35, 20 March 2013 (UTC)
 * Done, I think. Hopefully you'll have more of an idea what you're doing than I do. *shifty eyes* -— Isarra ༆ 20:15, 21 March 2013 (UTC)

Matma Rex (Bartosz Dziewoński) for mediawiki/core
I'm a pretty new developer (first gerrit patch merged in August last year), but I'm a (Polish) Wikipedia editor since 2007 and active gadget&template writer and an all-around tech guy since about 2009.

My forte is frontend (CSS and JS), and this is reflected in my commits and reviews. We have a huge backlog in this area, and I hope I'll be able to help.

Changes I was a reviewer of &bull; Bugzilla activity

I'm aware I have little experience compared to most of other core maintainers, but I thought, why not? Both positive and negative comments welcome :) Matma Rex (talk) 00:24, 24 March 2013 (UTC) Well, the family medical stuff dragged on a lot longer than I wish it had, and it's not over yet. I strongly regret the delay.
 * Just a note from Sumana here -- I'm notifying our community now and would like to wait a week before checking for consensus, just to give some folks time to look at Matma Rex's commits and reviews. (In my experience it usually takes several days anyway for people to think and put down their thoughts; I just figured I'd actually note it here so people have some kind of expectation regarding timeframe.)  Thanks for your work, Bartosz. Sharihareswara (WMF) (talk) 21:56, 25 March 2013 (UTC)
 * Weak . Weak because I've had limited opportunities to interact with him, +1 because they've always been very positive. Yuvipanda (talk) 22:00, 25 March 2013 (UTC)
 * I was actually thinking about nominating you. Bawolff (talk) 22:03, 25 March 2013 (UTC)
 * Useful insights and active in helping other contributors by giving a first review (thus saving work). But in my opinion still too inexperienced to be reviewing with CR+2 for merge. Krinkle (talk) 23:29, 25 March 2013 (UTC)
 * : Ideally nobody should be merging patches without a second opinion. I trust Matma Rex to know what and what not to merge. --MZMcBride (talk) 02:50, 26 March 2013 (UTC)
 * : I think MatmaRex will know how to use it responsibly. -- Krenair (talk &bull; contribs) 16:39, 26 March 2013 (UTC)
 * Support revoked for the same reasons as MarkTraceur. -- Krenair (talk &bull; contribs) 18:04, 16 April 2013 (UTC)
 * : If bunnies are allowed, why not him? --Nikerabbit (talk) 19:34, 26 March 2013 (UTC)
 * per MZMcBride --Waldir (talk) 19:54, 26 March 2013 (UTC)
 * The skill and commitment is there, but his manner of expressing criticism is scathing and combative and filled with contempt for anything he doesn't like. Maybe in the future, but not now. --Ori.livneh (talk) 04:29, 28 March 2013 (UTC)
 * I chatted with Bawolff about this on IRC, and he challenged me to provide examples of bad behavior in contexts other than the changes to the account creation interface. I was not able to find anything especially damning, which surprised me. It's possible that a combination of availability bias and feeling upset about recent interactions led me to believe this behavior was more pervasive than it in fact was.
 * I still believe that his behavior in the context of that discussion was sufficiently poor to merit waiting some time before revisiting this request, but as MZMcBride and Jack Phoenix note below, I also lost my cool in the end, and in doing so perhaps I forfeited the right to oppose the request on these particular grounds. So I am rescinding my veto. --Ori.livneh (talk) 05:03, 31 March 2013 (UTC)
 * Out of curiosity, what exactly did he do with ACUX that was such poor behaviour? -— Isarra ༆ 05:49, 31 March 2013 (UTC)
 * Saying things like "acux team should be slapped" and frantically pinging people on IRC to merge a revert (19:53:28 onwards). --Ori.livneh (talk) 06:06, 31 March 2013 (UTC)
 * Frantically? He pinged two people, Reedy and chad from what I can see... As for the slapping comment, That is pretty weak, I've pretty sure i've made worse against staff members, Oh and i've pinged more people to get code committed, merged or pushed to the live cluster. Peachey88 (talk) 08:07, 31 March 2013 (UTC)
 * He didn't ping me, the person who first +2'd it even though I was online at the time. (In his defense, he didn't know my handle and only found out after it was merged.) He also didn't ping anyone from the E3 team who wrote it even though they were online also. In fact, he picked two people in Platform who never even reviewed the original patch, and the three of the reasons for the revert were actually lifted from my review comments. I was actually reviewing the revert when Chad +2'd it--it was that fast (in fact, the only thing I had time to confirm was that it was a straight revert before it was already merged). By the way, I specifically waited until __after__ the Platform deploy to do the original +2 to give time for discussion to back out the change. As for whether or not he deserves +2, I don't have a negative opinion on that experience. I'm assume good faith (and given the font issue war (which I was aware of, but Matmarex and others were not aware that I was aware of)), I felt that it was just a miscommunication and Matmarex was unaware that I planned it so there would be two weeks to revise, amend, or revert before any code would have been deployed. I do hope that if he does get +2, he won't deliberately exploit the Features/Platform divide in the future…because if you want to go nuclear on "things that should not have been merged into mediawiki" all sides have a long list that it is helpful not to bring up. -Tychay (talk) 21:59, 11 April 2013 (UTC)
 * Yeah, generally speaking volunteers have to ping people to get things merged, often several people. Doesn't matter what it is. (That said, staff probably do similar, but perhaps in more organised ways about it since they're generally doing more specific things with specific teams and thus specific people would know about it/be following it and such.) -— Isarra ༆ 17:40, 31 March 2013 (UTC)
 * It isn't that he pinged people, it's who he pinged, the speed at which he did, and the way he represented the revert. I think the issue is calling it "a completely failurous patch" when it doesn't break mediawiki at all and never would have. That stings a lot given that I actually spent a lot of time doing the review and making sure it wouldn't break mediawiki and the parts I felt were sketch wouldn't be seen (were buried). I don't think he meant harm by doing so because he was probably panicking thinking I didn't do an adequate review (when I did), but it sure caused an unnecessary (and probably unintended) flair up. I hate to think that this is the only reason he won't get +2, because other than tone, I was ready to +2 his revert when Chad beat me to it. -Tychay (talk) 21:59, 11 April 2013 (UTC)
 * I could say the same of a number of people with +2 access, if you know what I mean. I share your concern, but I don't think it should preclude forward progress in development. If you think the skill and commitment are there and there are reasonable safeguards in place, it seems like it would be awfully difficult to oppose. For me, the benefit outweighs the (potential) cost. --MZMcBride (talk) 05:13, 28 March 2013 (UTC)
 * No, I don't know what you mean. If you see a current core committer behaving in a combative way, please call it out. --Ori.livneh (talk) 09:09, 28 March 2013 (UTC)
 * Seconding Ori: if there are other people with +2 powers in MediaWiki core who are often expressing themselves in scathing, combative ways, then we should ask these people to please act more collaboratively, and possibly even remove their privileges. I am willing to be the person who conveys complaints to such maintainers and tries to help them improve. Sharihareswara (WMF) (talk) 00:59, 30 March 2013 (UTC)
 * I'll just leave this, this and this here for you to draw your own conclusions. --Jack Phoenix (Contact) 00:24, 31 March 2013 (UTC)
 * If you worry that there is more to my objection than meets the eye, please state your concern explicitly, so that I may address it. --Ori.livneh (talk) 03:16, 31 March 2013 (UTC)
 * Would you describe adding (and self-merging) a snarky configuration variable as combative? I think the broader point here is a double standard (which everyone seems to acknowledge): one standard for Wikimedia Foundation staff/contractors and another (much stricter) standard for volunteers. I'm not sure this double standard is sustainable. --MZMcBride (talk) 03:42, 31 March 2013 (UTC)
 * I think you are wrong; can you please back up your assessment that everyone is acknowleding this double standard? I am a counterexample to your allegation, as I believe we should equally call out toxic behavior by staffers and by volunteers.  People should not act like jerks.  I hereby say: hey Ori, don't lose your cool like that in the future, thanks. Sharihareswara (WMF) (talk) 13:54, 20 April 2013 (UTC)
 * The way I read it, this allows ACUX stuff to be turned off if it is merged into core. -Tychay (talk) 21:59, 11 April 2013 (UTC)
 * for what it's worth I also support and ask for more civility. OrenB talk contib 04:38, 28 March 2013 (UTC)
 * - per MZMcBride. MatmaRex has also demonstrated an unusual impartiality, looking at the code itself and giving feedback based on what it is there rather than judging based on who submitted it - a valuable trait in a friend if you need someone to review your code, and, I would hope, a valuable trait in general when considering someone for core. -— Isarra ༆ 18:48, 28 March 2013 (UTC)
 * I rarely speak up in these discussions, but I have experience to contribute here. I'd like to see MatmaRex learn to improve how well he negotiates with others. Maintainers ought to be courteous and ought to give negative feedback in clear, but kind ways. His short-tempered comments have decreased others' morale and made it harder to solve problems together, despite the technical knowledge and enthusiasm he brings to MediaWiki. In code review, design discussions, and bug comments, those with +2 power have a special responsibility to see from others' points of view and avoid dismissiveness. I am happy to give more specific criticisms and suggestions upon request; MatmaRex and I have also spoken about these issues in the past. I do thank Bartosz for his work and encourage him to improve his negotiation skills and talk with us again in a few months. Sharihareswara (WMF) (talk) 00:59, 30 March 2013 (UTC)
 * While improved civility is a laudable goal, I have to wonder - is it a realistic one, or even an entirely applicable one in the situation? We have here a diverse community of paid staff and unpaid volunteers, operating on the idea that we can all contribute, but when there are so few volunteers with the ability to review and merge in core, and so few staff willing or even able to take the time to review volunteer submissions, it only makes more poignant the double standard that is apparently being applied to volunteers here. Yes, MatmaRex can be blunt, combative, and at times downright difficult, but the same and worse could be said of many staff members who had to go through no such vetting process for their access (yes, there was a different process, but expecting more of volunteers than of staff here of all places just seems ridiculous when there's a concern about chasing people off). But while some staffers have used their +2 for such when they really oughtn't have, I don't believe MatmaRex would do that - he already displays care with his regular reviews, only giving things a +1 or -1 if he is certain, and how he would use the added rights should be the important thing here, not his temperament, especially as the latter will affect matters regardless of his access. Core ownership, beyond the potential for harm, should not be a big deal - if a developer would be a net benefit to the project with access, he should have it. Do you believe MatmaRex, Bartosz Dziewoński, would not be a net benefit? -— Isarra ༆ 17:47, 30 March 2013 (UTC)
 * Can you please link to some of these comments you refer to? Peachey88 (talk) 05:24, 1 April 2013 (UTC)
 * I've not forgotten this request; due to some family medical stuff I've had some difficulties in the past week, but I will be giving some links this week. I'm sorry for the delay. Sharihareswara (WMF) (talk) 05:35, 10 April 2013 (UTC)

First, when you say "the same and worse could be said of many staff members who had to go through no such vetting process for their access" -- when I hear a complaint about a particular staff member's behavior, I have a word with them; if there's some kind of continuing pattern of toxic or condescending or uncollaborative behavior, I think it's worth bringing it up with me, with a request for revocation (or mentorship, as others have sugggested for MatmaRex).

I believe that yes, an offputting temperament has a bigger effect on the community if his privileges are increased; in my experience, if a person who makes short-tempered comments has more power, then that worsens the effects of their comments on the community, especially regarding newcomers.

I also think we have pretty different perspectives on the proportion of WMF staff who have made the time to review volunteers' contributions. Regardless, yes, it is a positive good to have more volunteers reviewing, and also great to have more volunteers with +2 in core+extensions or in more extensions. This is why I personally have initiated the conversation with several volunteers: "you, yes you, I bet you are ready to get core review power; do you want to self-nominate or shall I nominate you?" (hoo, bawolff, anomie (before he was hired), Tyler Romeo, Krenair, and others). And it's why I encouraged MatmaRex to seek +2 in l10n extensions via LevelUp.

When I notice that Bartosz has said something that hurts others' morale and makes community atmosphere worse, I generally bring it up with him in the moment; sometimes he responds with a fix, sometimes he calls the request for courtesy "political correctness". So my experience regarding his willingness to grow in this particular domain is mixed, and I'd welcome some thoughts from him on that. Ori and Terry already talked about the incident in which MatmaRex went about asking for a revert in a pretty anticollaborative way. I'd add that the way in which MatmaRex asks for review (sometimes pinging person after person to ask for a merge -- not for review, but for merge -- on a matter of no urgency) is difficult because of the context. It is of course not inherently wrong to ask for people to take a look at a backlogged change request sometimes. But, given that MatmaRex is not the only person looking for feedback and merge, and given that in these communications he sometimes conflates his own sense of urgency with what is highest-priority for MediaWiki as a whole, his behavior could be causing difficulty for the project. However this is a tricky matter and I'm very willing to discuss what our community customs ought to be given the current backlog; maybe everyone should be acting more like Bartosz, or maybe we should habitually demand that authors communicate clearly why their changesets require merge urgently, or maybe there is some other social norm we need to nurture. In February Bartosz said "It's incredible how it takes people more than two weeks to respond to such a simple question." I understand the frustration but there are ways to put that message that don't instantly spike the cortisol of everyone reading it and thus make the problem actually harder to solve. This pattern of dismissive or insulting off-the-cuff remarks -- calling things he doesn't care for "useless" or saying that the ACUX team "should be slapped" for using "fuckin' helvetica", for instance -- is one he has sometimes broken when I've told him to knock it off. (For example, in his "let me rephrase" comment and "re-reading my replies here"). And I have also seen his collaborative, willing-to-be-corrected, helpful side -- a lot! Which is why I have given him lots of advice, encouraged him, and thanked him for his work, and tried to help him improve as an engineer, which includes negotiation skills. I'm reasonably happy with his collegiality in the last month or so, and Mark Holmquist has offered to mentor him to help improve his behavior; if Bartosz says he's interested in that then I think the only question left to resolve is Krinkle's opposition based on Bartosz's experience level and skill. Sharihareswara (WMF) (talk) 13:54, 20 April 2013 (UTC)


 * Sumana, thank you for getting back to this. It's awesome to have more information to go on and I'm immensely happy to have had your two cents here. And, it should be said, thank you for helping to herd cats in our community. Everyone has to be reined in at one point or another, and you do a great job of helping us check our egos.
 * While I agree there are some rough edges around Bartosz's behaviour, I'm overall still impressed with him as a force for good. Some people just have a different way of interacting with the world, and while we'll work to help him be more encouraging, I don't think he's overly abrasive with people he doesn't know, which is good progress.
 * As a result I've reinstated my support. Thanks again for the thoughtful replies, Sumana, and I hope we can put this to bed soon :) --MarkTraceur (talk) 18:23, 20 April 2013 (UTC)

And regarding what MZMcBride wrote above, I'm sure we could use a few more volunteers with core +2 access; naturally such people need to be active and competent, and MatmaRex is both of those. --Jack Phoenix (Contact) 00:24, 31 March 2013 (UTC)
 * - per Ori and Sumana. --Jorm (WMF) (talk) 02:05, 30 March 2013 (UTC)
 * Revoking my position per Mark Traceur and Krenair.--Jorm (WMF) (talk) 18:48, 16 April 2013 (UTC)
 * Can you expand on your comment with more detail(s), after all, One the opposes you refer to has been striked. Peachey88 (talk) 05:22, 1 April 2013 (UTC)
 * Interesting. Only Wikimedia Foundation staff/contractors have opposed. --MZMcBride (talk) 21:06, 30 March 2013 (UTC)
 * But they're not the majority of the WMF staff/contractors who expressed an opinion, anyway, even if such counts happened to be meaningful. --Nemo 17:29, 31 March 2013 (UTC)
 * I obviously wholeheartedly this request. MatmaRex is a very competent developer who's not afraid to get his hands dirty and fix what others never bothered fixing (the Cologne Blue skin comes to mind, for example). It's also a good thing in my view if someone's willing to say how things really are, without the sugar coating. Sure, MatmaRex is capable of sharp and somewhat spiky opinions that some may not like, but...I've seen a lot worse &mdash; here and elsewhere. I think anyone who's been around for a bit longer has seen such things. I for one welcome harsh code criticism based on facts, as it's the only way to improve code quality.
 * . I have had good and bad interactions with Bartosz, but he's overwhelmingly a force for good, and he's getting much better. More experience interacting with the community, especially if the community supports and welcomes him, will cause him to get even better, even faster. And please, don't read into the fact that WMFers are going one way or another - we speak our minds, and some of us have spoken one way or the other on this one. The fact that volunteers haven't opposed this nomination doesn't necessarily mean anything. --MarkTraceur (talk) 00:56, 31 March 2013 (UTC)
 * I'm revoking my support temporarily because I think this procedure is falling apart and I don't want to see any decisions made until everyone has spoken their minds. I'm not opposing because I don't have all the evidence, but I'm no longer supporting because I don't have all the evidence. I am by no means convinced by either side right now, and I want to hear further evidence from both sides. --MarkTraceur (talk) 17:50, 16 April 2013 (UTC)
 * We've seen some further discussion on the topic and I don't think my convictions on the matter have particularly changed. I've reinstated my support, and I'll reply to Sumana's points above as well. --MarkTraceur (talk) 18:23, 20 April 2013 (UTC)
 * Conditional . I've had good experiences iterating on patch review in gerrit with Bartosz, and I think he could be a valuable team member. I would strongly recommend assigning a mentor -- my impression is that he may need more guidance on how to deal with people when reviewing others' code -- but we really should be doing this for all new folks. I can't promise particular response time myself, but will try to help out. Bonus points if someone is interested in volunteering for this! --brion (talk) 19:59, 8 April 2013 (UTC)
 * . He is motivated and seems to be figuring out the ropes of the MW dev community quickly after a bit of a rough start.  He's able to hang on despite valid criticism and improve. -- ☠ MarkAHershberger ☢ (talk) ☣ 20:05, 8 April 2013 (UTC)
 * Weak . I agree with Ori and Sumana, but I can see a path for improvement and eventual +2 with Bartosz. I like Brion's idea of mentorship, and I think that could work well. If mentored, and the concerns have been addressed, I don't see any reason we couldn't revisit this and I'd like to support at that time. ^demon[omg plz] 20:07, 8 April 2013 (UTC)
 * Striking because I can't really decide. Please consider me . ^demon</b><sup style="color:#c22">[omg plz] <i style="font-size:10px;">20:35, 8 April 2013 (UTC)</i>
 * From what I can see in his patches; he is mostly a positive force for getting things done. Even if he is a bit brusque in his dealings; I didn't really see egregiousness negativity beyond what is already expressed by other core developers. This is all something we can work on together but not, I feel, something that should stop us from acquiring another seemingly competent reviewer for CSS/JS. Mwalker (WMF) (talk) 20:32, 8 April 2013 (UTC)
 * There seem to be some accusations of Matmarex being "scathing". However, no evidence has been provided to support this, to the point where Ori has even rescinded his opposition. Furthermore, from what I've looked through in Gerrit of Matmarex's reviews, he seems to be perfectly fine when participating in discussion and has been assisting a lot with CSS/JS components. Considering the level of support he has received so far and the lack of evidence against him, I'm giving my support for this. If somebody is willing to provide evidence otherwise, I'll change my opinion. Parent5446 (talk) 21:00, 8 April 2013 (UTC)
 * My general feeling with Matmarex has been good. Entering the «discussion skills» topic, which is the only rejecting reason, I should note that the “offending review” was given to +2 people. While it is good to keep polite (and the case given doesn't seem too bad, more so for taking place in irc), it is sometimes more important to provide an accurate review. I wouldn't consider acceptable to tell a new contributor "You should be slapped for writing this code doing X", but it could be appropiate as a quick feedback (instead of not providing it at all) to eg. Tim Starling (of course, such bad code would probably end up being some clever trick outsmarting the reviewer, so you would better be damn sure of being right :). Don't misinterpret me, it is important to be nice and have a good feeling amongst developers, but I don't see the above evidence as a blocker for +2. Platonides (talk) 21:54, 8 April 2013 (UTC)
 * per Brion. I'm also too busy to be a mentor, but if one can be found that would be awesome. --Catrope (talk) 23:31, 8 April 2013 (UTC)
 * I think there is no reason to think that his behavior won't be more civil if he has +2 than currently. I suppose we should have a policy for recinding +2 access if I'm wrong (A good idea in general: I've been accused of deserving such a loss :-D). If he needs a mentor with +2, I can "volunteer" someone chill in Features (MarkTraceur or superm401) or maybe we can convince Krenair to do it. -Tychay (talk) 22:05, 11 April 2013 (UTC)
 * We do have one, at Gerrit/+2. Can someone link me to the rules about self-merging though, if I recall there was a discussion at one point whereby someone floated the idea that if you self-merge (unless in an emergency) you should lose your access (there were some exceptions IIRC too). The  helpful  one  22:11, 11 April 2013 (UTC)
 * Thanks, I didn't notice that that discussion was resolved. I think it was back in February. I don't think rules for self-merging were resolved other than it was frowned on. IIRC the discussion segwayed into a discussion of backporting patches. -Tychay (talk) 22:20, 11 April 2013 (UTC)
 * Terry says it as if it would be problematic, but so long as I can box out some time each week, I'd be happy to mentor Bartosz. --MarkTraceur (talk) 23:40, 11 April 2013 (UTC)

Jack Phoenix for mediawiki-skins
I've written, reverse-engineered and maintain(ed) plenty of skins (including, but most certainly not limited to the Nimbus skin (which is in the mediawiki/skins/ git repository) and several currently unreleased skins, such as Eminence, Hope and many more), and I've also contributed bugfixes to core skin stuff (PHP and CSS&JS parts). In addition to those, I've also tried to document how the scary internals work and I even attempted to rewrite the skin system back in the SVN era. Suffice to say, I believe it's clear that I have a good understanding of how the skin system works and enough experience to have project ownership. --Jack Phoenix (Contact) 21:26, 29 March 2013 (UTC)
 * I'm not sure this needs a vote, but I, of course. --MZMcBride (talk) 21:07, 30 March 2013 (UTC)
 * (semi-weak) . You've been quite a prolific contributor, not to mention you've been here for quite a long time - certainly much longer than me. My only concern is it appears you haven't committed anything since the git transition (Possible exception of 559c2a48c which doesn't show up in gerrit), and have only done a very minor amount of reviewing/commenting in gerrit (I found about 4 things you commented on). Nonetheless you're still someone I trust, hence the support. Bawolff (talk) 03:34, 31 March 2013 (UTC)
 * Absolutely. ^</b>demon</b><sup style="color:#c22">[omg plz] <i style="font-size:10px;">20:09, 8 April 2013 (UTC)</i>
 * Yeah, what they said; he knows his stuff. -— Isarra ༆ 21:50, 8 April 2013 (UTC)
 * Platonides (talk) 21:55, 8 April 2013 (UTC)
 * Of course. --Catrope (talk) 23:32, 8 April 2013 (UTC)
 * Skins is in extensions? Well then, I support if Catrope does. -Tychay (talk) 22:12, 11 April 2013 (UTC)
 * I totally got confused and thought this was for the skins directory in mediawiki/core. Bawolff (talk) 23:52, 11 April 2013 (UTC)

Jeroen for core
I believe it's high time that Jeroen has core access. He's the author of dozens of extensions, tons of core features (including ContentHandler), and is generally a great guy. I think he's grown a lot over the years since coming to us as a GSoC student, and it's shown in both the quality of his code and the review he gives others. He knows how everything works, and he's got a good eye for making sure things are well-tested.
 * As nominator. ^</b>demon</b><sup style="color:#c22">[omg plz] <i style="font-size:10px;">14:50, 17 April 2013 (UTC)</i>
 * Weak Another one of those "Wait, we...haven't yet?" --MarkTraceur (talk) 16:21, 17 April 2013 (UTC)
 * Edited to add "weak" - Krenair's note is a bit concerning. --MarkTraceur (talk) 18:28, 20 April 2013 (UTC)
 * per MarkTraceur Yuvipanda (talk) 16:32, 20 April 2013 (UTC)
 * I'm a bit unhappy that he has been merging his own changes, in violation of Gerrit/+2: (mouseover the CR column on most of those rows). There were some other concerns raised in a conversation I saw in an IRC channel two days ago. I can share those log extracts in private but I'm not sure how everyone else feels about them being public (it is a public channel though). -- Krenair  (talk &bull; contribs) 16:52, 20 April 2013 (UTC)