Tackling review culture

I’ll start with a clarification: This problem isn’t unique to Foreman

What problem? Oh, right, I haven’t defined it yet. So here’s the issue - sometimes, every few months or so, we hit a situation where on one (or more) of our repos, Pull Requests (PRs) are being forgotten about. This leads to problems - if they’re big PRs, it probably means something important isn’t getting done. If it’s small PRs, then likely, it’s useful bugfixes that aren’t getting merged. If the author is a core dev, then it’s demoralising, and they may leave the project. If it’s a new contributor, the experience may put them off further contribution. It’s not good, any way you slice it.


This is a companion discussion topic for the original entry at https://theforeman.org/2015/11/tackling_review_culture.html