Junio C Hamano <gits...@pobox.com> writes:
> Felipe Contreras <felipe.contre...@gmail.com> writes:
>> diff --git a/contrib/related/git-related b/contrib/related/git-related
>> new file mode 100755
>> index 0000000..b96dcdd
>> --- /dev/null
>> +++ b/contrib/related/git-related
>> @@ -0,0 +1,124 @@
>> +#!/usr/bin/env ruby
>> +# This script finds people that might be interested in a patch
>> +# usage: git related <file>
>> +$since = '5-years-ago'
>> +$min_percent = 10
>> +def fmt_person(name, email)
>> + '%s <%s>' % [name, email]
> Micronit. I suspect you do not need this helper, unless later
> patches start using it.
Not that matters terribly, but "later patches start using it in a
different way" may be needed as a clarification.
The current two callers capture both "%s <%s>" as two separate
groups but they do not need to; they can do "(%s <%s>)" and pass $1
to this function, I think.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html