Skip to Content

Re: Reciprocity in PR opening vs reviews; banning contributors

> Holger, what do you make of this inverted carrot approach rather than a

> stick?

seeing it fail for 10+ years


-- 
Your partner for the hard Odoo problems
https://hunki-enterprises.com

by Holger Brunn - 09:50 - 27 Jan 2026

Reference

  • Reciprocity in PR opening vs reviews; banning contributors
    Hi all,
    
    on the OCA days, we discussed that the current situation with having way too 
    much input for way too few reviewers is untenable. This has not improved 
    since, quite the opposite. It's really hard to find the gems in the noise.
    
    Back then, I called for better automation for this, so here my proposal:
    
    Have a github action that counts lines of PRs somebody opened in a repo, vs 
    the lines of PRs the person reviewed in that repo. Everyone must review at 
    least twice as much as they submit.
    
    If after asking for more reviews, no reviews come, close the user's PRs 
    automatically (in the repo, not all OCA) after some time.
    
    Also add a manual mechanism for banning users who try to cheat with bullshit 
    reviews or otherwise undesirable behavior. PRs by banned users are closed 
    automatically.
    
    I implemented both in
    https://github.com/hbrunn/social/blob/18.0/.github/workflows/reciprocity.yml
    resp
    https://github.com/hbrunn/social/blob/18.0/.github/workflows/ban.yml
    
    You can test this by creating PRs against my fork after cloning my version of 
    the 18.0 branch.
    Banning works by adding a handle to a file .banned.txt in the repo's root.
    
    Before proposing this to oca-addons-repo-template, I'd like to hear some input 
    from you.
    
    Best regards,
    Holger
    
    
    -- 
    Your partner for the hard Odoo problems
    https://hunki-enterprises.com

    by Holger Brunn - 08:56 - 23 Jan 2026