From 9361a5b865c720bcc062abf951f52688bc235079 Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Tue, 28 Apr 2026 14:28:36 +0200 Subject: [PATCH] docs: describe the contributions workflow for RADV MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This workflow has been discussed a lot with the team for the past few years. Let's just clarify it for real in the documentation. Co-written-by: Timur Kristóf Signed-off-by: Samuel Pitoiset Part-of: --- docs/drivers/radv.rst | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/docs/drivers/radv.rst b/docs/drivers/radv.rst index 8cf8dbeef5c..60f2a1a8b1d 100644 --- a/docs/drivers/radv.rst +++ b/docs/drivers/radv.rst @@ -18,6 +18,39 @@ The easiest way to track the feature set of RADV (and other Vulkan drivers in Me take a look at the `Mesa matrix `__. +Contributing to RADV +~~~~~~~~~~~~~~~~~~~~ + +RADV is part of Mesa, so we recommend reading Mesa's +guidelines for :doc:`submitting patches <../submittingpatches>`. + +Additionally, the RADV team agreed on the following workflow for contributions: + +* Do NOT merge any MRs without at least one approval from someone familiar with + the code. +* There are different types of approvals: + + - Reviewed-by tag + + - MR approval button + + - Acked-by tag + +* After receiving the first approval, please wait at least 24 hours (excluding + weekends) before merging the MR. This is to make sure that anyone who may be + interested had opportunity and time to look at the MR (eg. timezone + differences). The wait is not required in the following cases: + + - The MR is trivial (eg. very simple cleanups, typos, CI flakes updates) + + - The MR has been approved by two or more developers + +* Even if your MR has been approved and is ready to be merged, please make sure + to resolve any open conversations about potentially controversial commits. + +In case your MR needs to be merged immediately (should be rarely needed), feel +free to ping the developers to get more approvals. + Supported hardware ~~~~~~~~~~~~~~~~~~