<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
>         Executing the event means executing ``action(*argument, **kwargs)``.<br>
>     -   *argument* must be a sequence holding the parameters for *action*.<br>
>     -   *kwargs* must be a dictionary holding the keyword parameters for *action*.<br>
>     +   Optional *argument* argument must be a sequence holding the parameters<br>
>     +   for *action* if any used.<br>
>     +   Optional *kwargs* argument must be a dictionary holding the keyword<br>
>     +   parameters for *action* if any used.<br>
><br>
><br>
> I don't see how this change improves the documentation. To keep the grammar<br>
> correct and just state that the arguments are optional, I would simply replace<br>
> "must be" by "is". For example:<br>
><br>
>   *argument* is a sequence holding the parameters for *action*.<br>
><br>
> This is short, and since the function signature clearly shows that argument has<br>
> a default value, I think it conveys the meaning it should.<br>
<br>
</div></div>Hi Eli,<br>
<br>
I'm sure we non-native speakers are fine with any improvements you can make<br>
during commit review.<br>
<br>
Georg<br></blockquote><div><br></div><div>Georg,<br><br></div><div>I also wrote a private email to Serhiy proposing to help, but since you brought this up here: I think that my comment was constructive. What should have I done differently? Go ahead and modify the phrasing in a separate commit? I see a couple of problems with that:<br>

<br></div><div>1. It can be somewhat disrespectful to a new committer, and I wanted to reach a consensus first.<br></div><div>2. Serhiy diligently committed this into 3 or 4 different Python branches. With all due respect, going through the merge/push dance is far above the effort I'm willing to invest in this.<br>

<br></div><div>Eli<br><br></div><div>P.S. I would argue that you are more native-speaker than myself w.r.t. English :-)<br><br><br></div><div><br><br></div><div> </div></div></div></div>