<div dir="ltr">Thanks for mentioning the contributor guide!<div><br></div><div>I'll happily review any patches you have for that section. I'm sure Ildiko would be happy to as well. </div><div><br></div><div>-Kendall (diablo_rojo)</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 12:41 PM William M Edmonds <<a href="mailto:edmondsw@us.ibm.com">edmondsw@us.ibm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><p><tt><font size="2">Jeremy Stanley <<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</a>> wrote on 12/05/2018 02:52:28 PM:<br>> On 2018-12-05 14:48:37 -0500 (-0500), William M Edmonds wrote:<br>> > Eric Fried <openstack@fried.cc> wrote on 12/05/2018 12:18:37 PM:<br>> > <br>> > <snip><br>> > <br>> > > But I want to edit 1b2c453, while leaving ebb3505 properly stacked on<br>> > > top of it. Here I use a tool called `git restack` (run `pip install<br>> > > git-restack` to install it).<br>> > <br>> > It's worth noting that you can just use `git rebase` [1], you don't have to<br>> > use git-restack. This is why later you're using `git rebase --continue`,<br>> > because git-restack is actually using rebase under the covers.<br>> > <br>> > [1] <a href="https://stackoverflow.com/questions/1186535/how-to-modify-a-" target="_blank">https://stackoverflow.com/questions/1186535/how-to-modify-a-</a><br>> specified-commit<br>> <br>> You can, however what git-restack does for you is figure out which<br>> commit to rebase on top of so that you don't inadvertently rebase<br>> your stack of changes onto a newer branch state and then make things<br>> harder on reviewers.<br>> -- <br>> Jeremy Stanley<br></font></tt><br></p></div><div><p><tt><font size="2">Ah, that's good to know.</font></tt><br><br><tt><font size="2">Also, found this existing documentation [2] if someone wants to propose an update or link from another location. Note that it doesn't currently mention git-restack, just rebase.</font></tt><br><br><tt><font size="2">[2] </font></tt><a href="https://docs.openstack.org/contributors/code-and-documentation/patch-best-practices.html#how-to-handle-chains" target="_blank"><tt><font size="2">https://docs.openstack.org/contributors/code-and-documentation/patch-best-practices.html#how-to-handle-chains</font></tt></a><br>
</p></div>
</blockquote></div>