I had a trivial PR (literally just a rubygem update) for a darwin package take 22 days to review, including posting about it twice in the “PRs ready for review” thread. @worldofpeace finally merged it for me yesterday, and commented that this likely took so long due to the small number of macos reviewers.
To that end, I’d like to propose a GitHub label be added to darwin-only packages, to make it easier for macos reviewers to focus on the packages that other people can’t review. Furthermore, there should be a publicly-available bot instruction that will add this label to PRs where ofborg doesn’t detect it automatically (in particular, bundlerApp PRs frequently don’t modify the default.nix file and therefore ofborg doesn’t detect the derivation).
You’re right in that there is a label there. I’m curious what criteria ofborg uses to attach it; looking at all open PRs with that label, nearly all have “darwin” or “macos” in the title. The one that doesn’t, the label was added manually (by you).
So I guess we need to update the bot to attach it to any PR whose derivation declares darwin as the sole platform, and as previously suggested it would be great to have a publicly-available bot command to request it to be added to packages where it’s not detected.
Right, which is why I want to make it easier for Darwin contributors to find packages that need their attention.
Speaking personally, I’m maintaining a number of packages, and would be willing to do some periodic reviewing of other people’s packages (no commit bit but I can at least check them), but I generally don’t because I’m not sure what packages it’s worth me looking at on macOS. Now that I know about the Darwin label I can at least look at those, but when looking right now I don’t see anything obvious that looks like a trivial PR that’s languishing due to lack of review.
Beyond the fact that it’s probably not appropriate to ping a complete team of maintainers just to expedite darwin package review, it also appears that this team is not publicly-visible.
