<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 7, 2016 at 9:48 PM, Brett Cannon <span dir="ltr"><<a href="mailto:brett@python.org" target="_blank">brett@python.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><br><div class="gmail_quote"><span class=""><div dir="ltr">On Mon, 6 Jun 2016 at 13:20 Anish Shah <<a href="mailto:shah.anish07@gmail.com" target="_blank">shah.anish07@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-family:tahoma,sans-serif">Hello everyone,</div><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif">This mail is about my progress so far in integrating GitHub and few doubts/questions about how to do stuff.</div><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif"><b>Progress</b></div><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif"><b>Show GitHub Pull Request comments on b.p.o</b>. [1] - Issue and review comments will be added to the linked b.p.o issue. Only one comment will be posted over 30 minutes. </div><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif"><b>Questions/Doubts</b></div><div style="font-family:tahoma,sans-serif"><b><br></b></div><div style="font-family:tahoma,sans-serif"><b>Converting patches to pull requests </b>- First, let me tell you all my approach and questions that I have. Please let me know if you have better ways to do this.</div><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif"><b>Approach</b></div><div style="font-family:tahoma,sans-serif">We need to have a git binary on b.p.o server and when a patch is uploaded, we run a series of git commands from roundup to apply the patch, commit using contributor's credentials, push the new branch and finally create a new pull request using GitHub API.</div><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif"><b>Questions</b></div><div style="font-family:tahoma,sans-serif"><b><br></b></div><div style="font-family:tahoma,sans-serif">1. How should we inform the contributor about the newly created PR? (S)he won't get subscribed to it automatically as the PR is created using API. </div></div></blockquote><div><br></div></span><div>A mention in the issue and in a GitHub comment should be enough.<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif">2. What should we do an issue affects multiple versions? Should we open multiple pull requests against each branch?</div></div></blockquote><div><br></div></span><div>We have not decided how we want to handle this in the workflow.<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif">3. There can be different patches for 2.7 and 3.x versions. Currently, there's no way of determining which patch is for which branch. So, it would be difficult to create a new pull request.</div></div></blockquote><div><br></div></span><div>Yep, that's tricky.<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif">Looking forward to what you guys think.</div></div></blockquote><div><br></div></span><div>Obviously what you work on is between you and Maciej, Anish, but I view this feature as low-priority. To me the two most important things are tying a pull request to an issue (I know you added a field but we will also need some GitHub web hook that will detect an issue in a PR title and automatically make the connection), and leaving a comment on an issue once a commit lands that mentions an issue in the commit message (like we currently do w/ hg).<br></div></div></div></blockquote><div><br></div><div><div class="gmail_default" style="font-family:tahoma,sans-serif">​Thanks Brett. I have already submitted an initial patch for linking a pull request to an issue using GitHub web hooks. :) ​</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></div><div><br></div><div>-Brett<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><div dir="ltr"><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif">Thanks, </div><div style="font-family:tahoma,sans-serif">Anish Shah </div><div style="font-family:tahoma,sans-serif"><br></div><div style="font-family:tahoma,sans-serif">[1] <a href="http://psf.upfronthosting.co.za/roundup/meta/issue592" target="_blank">http://psf.upfronthosting.co.za/roundup/meta/issue592</a></div></div></span>
_______________________________________________<br>
core-workflow mailing list<br>
<a href="mailto:core-workflow@python.org" target="_blank">core-workflow@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/core-workflow" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/core-workflow</a><br>
This list is governed by the PSF Code of Conduct: <a href="https://www.python.org/psf/codeofconduct" rel="noreferrer" target="_blank">https://www.python.org/psf/codeofconduct</a></blockquote></div></div>
</blockquote></div><br></div></div>