<div dir="ltr"><div><div><div>> After pushing my changes, the sanity bot found some typos in my commit 
message. Now, how would I proceed? Do I change the commit message? If 
so, how would I do that?<br><br></div>IIRC you do your modifications, do a git commit --amend, keep the same code review ID in the commit message and push the new set of changes<br></div><div><br></div><div>>  How could I find reviewers for my changes to QTextDocumentWriter?</div><div><br></div><div><a href="https://wiki.qt.io/Maintainers">https://wiki.qt.io/Maintainers</a></div><div><br></div><div>you can also come and ask on IRC<br></div><div><br></div>Best,<br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div><div><font color="#783f04"><br></font></div><div><font face="arial, helvetica, sans-serif" size="2" color="#134f5c">-------</font></div><font face="arial, helvetica, sans-serif" size="2" color="#134f5c">Jean-MichaĆ«l Celerier</font><div><font face="arial, helvetica, sans-serif" size="2" color="#134f5c"><a href="http://www.jcelerier.name" target="_blank">http://www.jcelerier.name</a></font></div></div></div></div>
<br><div class="gmail_quote">On Wed, Oct 4, 2017 at 8:55 AM, Daniel Savi <span dir="ltr"><<a href="mailto:daniel.savi@gaess.ch" target="_blank">daniel.savi@gaess.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello everybody<br>
<br>
I've just pushed my first commit to QtGui, trying to follow the contribution guidelines posted here <a href="http://wiki.qt.io/Qt_Contribution_Guidelines" rel="noreferrer" target="_blank">http://wiki.qt.io/Qt_Contribut<wbr>ion_Guidelines</a>. Now I have some questions regarding the process. It seems that I have done it at least partly wrong ;-)<br>
<br>
The codereview to my changes is as follows: <a href="https://codereview.qt-project.org/#/c/207540/" rel="noreferrer" target="_blank">https://codereview.qt-project.<wbr>org/#/c/207540/</a><br>
<br>
After pushing my changes, the sanity bot found some typos in my commit message. Now, how would I proceed? Do I change the commit message? If so, how would I do that?<br>
<br>
Another probably more serious problem: I've checked out the source for 5.10 and created a branch "myfix5.10" and used "git checkout myfix5.10". Then I mad a diff to my locally changed files in another directory and patched the files from 5.10 in a temporary folder. After checking that the patched files looked how they should, I copied them back to the original folder. Then I did a "git commit -a". When checking the branch with "git branch", I found that I was on a detached head now. I couldn't go back to the branch, because git told me that I would lose my changes. So I pushed them anyway. Now, my submit type in Gerrit is "cherry pick". What should I have done, when encountering the detached head state? Is the "cherry pick" type a problem?<br>
<br>
Third and last, I didn't know how to find reviewers so I picked some almost randomly from the git log and added the QtGui maintainer, too. Probably not such a good idea? How could I find reviewers for my changes to QTextDocumentWriter?<br>
<br>
Sorry for the lengthy post.<br>
<br>
______________________________<wbr>_________________<br>
Development mailing list<br>
<a href="mailto:Development@qt-project.org" target="_blank">Development@qt-project.org</a><br>
<a href="http://lists.qt-project.org/mailman/listinfo/development" rel="noreferrer" target="_blank">http://lists.qt-project.org/ma<wbr>ilman/listinfo/development</a><br>
</blockquote></div><br></div>