[FFmpeg-devel] [PATCH 1/2] doc/developer: Reviews must be constructive
Vittorio Giovara
vittorio.giovara at gmail.com
Fri Aug 25 04:56:44 EEST 2023
On Thu, Aug 24, 2023 at 9:56 PM Michael Niedermayer <michael at niedermayer.cc>
wrote:
> Suggested text is from Anton
>
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
> doc/developer.texi | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/doc/developer.texi b/doc/developer.texi
> index 0c2f2cd7d1..383120daaa 100644
> --- a/doc/developer.texi
> +++ b/doc/developer.texi
> @@ -853,6 +853,9 @@ Everyone is welcome to review patches. Also if you are
> waiting for your patch
> to be reviewed, please consider helping to review other patches, that is
> a great
> way to get everyone's patches reviewed sooner.
>
> +Reviews must be constructive and when rejecting a patch the reviewer must
> explain
> +their reasons and ideally suggest an alternative approach.
>
NAK
we shouldn't put extra burden on reviewers, nor guilt trap them into
suggesting an alternative approach
offlist and irc discussion is of course recommended, but writing this rules
in stone will only deter good reviews, in my opinion
--
Vittorio
More information about the ffmpeg-devel
mailing list