<div dir="ltr">Hi,<div><br></div><div>Having made the changes as per suggestions, I'd like to draw your kindest attention to my PR and request you for reviewing the same. Link to the PR: <a href="https://github.com/macports/macports-ports/pull/6784">https://github.com/macports/macports-ports/pull/6784</a>.</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="color:rgb(80,0,80)">With Warmest Regards,</div><div style="color:rgb(80,0,80)">Yours Sincerely,</div><div dir="ltr" style="color:rgb(80,0,80)"><i>Saptarshi Mukherjee,</i><div><i>A GSoC 2020 Aspirant.</i></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 7, 2020 at 3:40 PM SAPTARSHI MUKHERJEE <<a href="mailto:saptarshim@iitbhilai.ac.in">saptarshim@iitbhilai.ac.in</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Thank you Sir. I'll keep them in mind.<div><br clear="all"><div><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div>With Warmest Regards,</div><div>Yours Sincerely,</div><div dir="ltr"><i>Saptarshi Mukherjee,</i><div><i>A GSoC 2020 Aspirant.</i></div></div></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 7, 2020 at 2:39 PM Mojca Miklavec <<a href="mailto:mojca.miklavec.lists@gmail.com" target="_blank">mojca.miklavec.lists@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, 7 Apr 2020 at 06:23, SAPTARSHI MUKHERJEE wrote:<br>
><br>
> May kindly validate the final commit, so that I may rebase the commits with the right message and you may merge the same.<br>
<br>
Why do you wait to get approval before fixing "a known issue"?<br>
GitHub offers an "Approve" button that developers may click without<br>
leaving any further feedback, but this one cannot be clicked before<br>
the contribution is ready to be merged; it's worse, one could have<br>
clicked "Request changes" and then you'll be stuck with a red cross<br>
until that same person checks again.<br>
<br>
> P.S.: If I'm undesirably spamming all of you, I'm sincerely apologetic about it. May please let me know if I should mail these to some specific recipients instead of the mailing list.<br>
<br>
I would advise against mailing individual recipients unless it's<br>
really off-topic and you know whom to contact or when someone might<br>
have missed the email in the sea of other messages from the mailing<br>
list.<br>
<br>
Generally the communication for such issues would be kept on the<br>
GitHub inside a code review, but asking the mailing list for attention<br>
if you get stuck somewhere, or if nobody has responded for a couple<br>
days.<br>
<br>
At these circumstances when we asked you for contributions it's<br>
probably ideal to send one email when you have opened a new PR to<br>
avoid a too long delay in response. When people have provided feedback<br>
to you, they will likely get a notification when you change something<br>
almost immediately, and there's certainly no need to send multiple<br>
reminders per day with requests to check your changes.<br>
<br>
If you are bored while waiting for a response, start working on a<br>
different ticket :)<br>
<br>
Mojca<br>
</blockquote></div>
</blockquote></div>