Re: [FFmpeg-devel] [PATCH] tools: add general_assembly.pl

["J, Dekker" <jdek@itanimul.li>]

On 15/02/2022 17:46, Jean-Baptiste Kempf wrote:
> Hello,
> 
> On Tue, 15 Feb 2022, at 12:50, J. Dekker wrote:
>>   This was rejected last time but I would really like to get this in the
>>   tools or at least publicly recorded on mailing list since the script
>>   was updated.
> 
> Why was this rejected?

It was blocked by Nicolas as the procedure to determine the general 
assembly was not fully decided.

>> +        foreach my $commit (@commits) {
>> +            if ($commit =~ /\n[\w\/]+\.(c|h|S|asm)/) {
>> +                $true++;
>> +            }
> 
> Why do we filter on those file types? .md, .pl and other things for docs are active in the community.

It was just discussed at the in person meeting, our documentation states 
otherwise and I think we should stick to the documentation here. It 
doesn't make that much sense to filter out people who have majority 
documentation commits either--20 documentation commits is still a 
significant enough contribution.

See updated patch.

-- 
J. Dekker
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".