On 10/12/22 14:11, Alex Bennée wrote:
Hi,
This is an attempt to improve our processes documentation by:
- adding an explicit section on maintainers
- reducing the up-front verbiage in patch submission
- emphasising the importance to respectful reviews
I'm sure the language could be improved further so I humbly submit
this RFC for discussion.
Alex Bennée (4):
docs/devel: add a maintainers section to development process
docs/devel: make language a little less code centric
docs/devel: simplify the minimal checklist
docs/devel: try and improve the language around patch review
docs/devel/code-of-conduct.rst | 2 +
docs/devel/index-process.rst | 1 +
docs/devel/maintainers.rst | 84 +++++++++++++++++++
docs/devel/submitting-a-patch.rst | 101 +++++++++++++++--------
docs/devel/submitting-a-pull-request.rst | 12 +--
roms/qboot | 2 +-
6 files changed, 157 insertions(+), 45 deletions(-)
create mode 100644 docs/devel/maintainers.rst
Thanks, these are useful improvements. On top we could probably merge
some content from Linux and make the documentation standalone. But still:
Reviewed-by: Paolo Bonzini <pbonz...@redhat.com>
after addressing comments from Stefan and myself.
Paolo
