[FFmpeg-devel] [PATCH 2/2] doc/developer: Code pushed without patches on ffmpeg-devel must be announced on the ML

Nicolas George george at nsup.org
Fri Aug 25 20:42:31 EEST 2023


James Almer (12023-08-25):
> As i said on IRC,

A medium that can only be accessed by a small fraction of the
contributors due to timeliness constraints.

>		    there's a difference between "Unless justified, please
> send things to the ML before pushing" and "Everything must have at least two
> reviews from long standing developers".
> 
> This is not requiring everything to have been reviewed before being pushed,
> it's asking for non trivial things to hit the ML so people have at least a
> chance at finding problems.

Faire enough. Just let us make it very clear it is the former and not
the latter. The I-don't-know-the-code-“LGTM” mandatory review were a
disgrace.

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20230825/b84ae5aa/attachment.sig>


More information about the ffmpeg-devel mailing list