Mail archive

[alpine-devel] How to improve quality control for patch reviews

From: Natanael Copa <>
Date: Thu, 26 Jul 2018 14:15:58 +0200


We have had a few complaints from Jakub Jirutka about qualtiy control.

A recent example is

We do struggle with keep up with the PR queue and at the same time do
good enough quality control. I also think we may have different
opinions what is good enough.

I wonder if you have any ideas how we deal with this?

A few things I think we can do:

- improve documentation. Write documentation with a simple checklist
  you can look over before you submit a PR. For example, "check that
  license is in SPDX format[1]", "check that it does not automatically
  start services from pre-install", etc. This will make it easier for
  people doing patch reviews and can be useful when adding automated

- add more automatic checks

- give more people push access. Look for people that are candidates to
  get push access. Help them to improve. Follow up when they are "good

Any other ideas or thoughts?



Received on Thu Jul 26 2018 - 14:15:58 UTC