<div dir="ltr">A simple way to proceed would be to advise submitters to submit their "atomic" PRs in a valid order. If change B depends on change A, then submit A first, wait until merged, and then submit B. This would have the nice side effect of not overwhelming the integrators.</div><div class="gmail_extra"><br><div class="gmail_quote">2018-03-01 15:04 GMT+01:00 Vaclav Hapla <span dir="ltr"><<a href="mailto:vaclav.hapla@erdw.ethz.ch" target="_blank">vaclav.hapla@erdw.ethz.ch</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><br><div><br><blockquote type="cite"><div>1. 3. 2018 v 14:04, Patrick Sanan <<a href="mailto:patrick.sanan@gmail.com" target="_blank">patrick.sanan@gmail.com</a>>:</div><span class=""><br class="m_5325355381168353465Apple-interchange-newline"><div><div dir="ltr">Maybe it would also help to add more explicit instructions to the wiki (<a href="https://bitbucket.org/petsc/petsc/wiki/Home#markdown-header-contributing-to-petsc" target="_blank">https://bitbucket.org/petsc/<wbr>petsc/wiki/Home#markdown-<wbr>header-contributing-to-petsc</a>) on how to construct a branch that is likely to get through the integration steps quickly.<br><br>I'd suggest adding language about these (and volunteer to write it), even if some might be obvious:<br>- Adding tests for whatever you submit<br>- Testing with configurations other than the usual double/real/C setup (complex, single) <br>- Making the PR as small/atomic as possible (can your PR be 2 or more separate PRs?)</div></div></span></blockquote><div><br></div><div><div>Yes, I think it's quite common that one would like to factor out one or more smaller bugfixes and/or improvements from a bigger PR. But some of them may depend on some others, and definitely the original big PR depends on all of them. Perhaps it would be nice if there is some documented way of specifying dependendencies between PRs to insure a proper order of merging (I think BitBucket has no such feature?).</div><div><br></div><div>Thanks,</div><div>Vaclav</div><div><br></div></div><span class=""><blockquote type="cite"><div><div dir="ltr"><div>- Running through valgrind (using --download-mpich) before submitting</div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2018-03-01 12:33 GMT+01:00 Karl Rupp <span dir="ltr"><<a href="mailto:rupp@iue.tuwien.ac.at" target="_blank">rupp@iue.tuwien.ac.at</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Dear PETSc folks,<br>
<br>
I think we can do a better job when it comes to handling pull requests (PRs). We have several PRs piling up, which after some time (imho) get merged relatively carelessly instead of reaping the full benefits of a thorough review.<br>
<br>
In order to improve the integration of pull requests, I propose to nominate a PR integrator, who is a-priori responsible for *all* incoming PRs. The PR integrator is free to delegate a particular PR integration to someone with the relevant domain-specific knowledge (e.g. Matt for DMPlex-related things) by appropriate comments on Bitbucket. In case of delays, the PR integrator is also responsible for issuing reminders over time (like Barry has done in the past).<br>
<br>
The idea is to make daily progress with the PRs. One integration step per day (e.g. testing or merging to next) is presumably enough to handle the load, whereas things get messy if we let things pile up. Automated testing may help a bit in the future, but it doesn't release us from properly reviewing the contributed code. </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Any objections to my PR integrator proposal? Any volunteers? ;-)<br>
If nobody else wants to be the highly esteemed PR integrator, I can do it. ;-)<br>
<br>
Best regards,<br>
Karli<br>
</blockquote></div><br></div></div></div></div></div>
</div></blockquote></span></div><br></div></blockquote></div><br></div>