Define code review guidelines for Python codebase

Fred de Gierrequested to merge
507548-define-code-review-guidelines-for-python-codebase into master

What does this MR do and why?

This update adds comprehensive Python code review guidelines to the maintainership documentation. It introduces five key review focus areas: code style, code quality, testing, documentation, and security - each with specific checkpoints for reviewers. The changes also recommend automated tools for ensuring code quality, including formatting tools (Black, isort), linting tools (flake8, pylint, mypy), testing tools (pytest), and security scanning tools. To help teams implement these practices, the update includes example configurations for both a Makefile with common development commands and a GitLab CI configuration that automates linting and testing in the CI pipeline.

References

Screenshots or screen recordings

Before After

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #507548 (closed)

Edited by Fred de Gier

Merge request reports