Code review/Patch board

Often people complain that their patches fall through the cracks or that it can be really difficult to get reviews for some patches. This is an experimental process meant to address that. It is experiment, so may change or be discarded.

General rules
This is an experimental process, so consider these more guidelines than hard and fast rules
 * Each person is only allowed to list a single patch at one time
 * You can remove your patch to add a different one, but if you do that your new patch has to go to the bottom of the list
 * New patches are placed at bottom of list
 * If you list a patch, then you are taking responsibility for addressing any code review. Generally you should only list your own patches or patches you have taken over.
 * You can only list patches here that have had no meaningful activity on gerrit for at least a week, at time of listing. Meaningful activity means the patch had non-trivial changes or feedback from others. Minor changes to a commit message or rebasing does not count as meaningful.
 * Your patch should either apply cleanly or have been rebased relatively recently (e.g. within the last 2 weeks).
 * Your patch must be passing all jenkins checks before being listed here and must not have any outstanding -1's at time of listing.
 * Once someone reviews your patch (Either +2's or -1's it) it goes off the list. Reviewers are encouraged to re-review patches once the original contributor fixes the issues. If the patch gets stuck again, it is also allowed to go back on the list
 * For now, only patches to mediawiki core or a WMF deployed extensions. In future maybe we'll expand to other things if this goes well.
 * No patches that are controversial - whether from an architectural or user perspective. This process is not a replacement for TDF/TechComm, phabricator or a wikitech-l post. If there is active disagreement about whether the thing the patch is doing is a good idea in principle, you have to get consensus on that before bringing it here.
 * It is generally discouraged to bring patches here that are marked XL or bigger by the gerrit size criteria. Try to make your patches more bitesized. That said this is not a hard rule (Does this make sense?)

Patches

 * Add your patch here!
 * martineznovo: - PoolCounterRedis: Fix wrong cast, locks weren't being released. --Ciencia Al Poder (talk) 18:07, 1 February 2023 (UTC)