<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 16, 2017 at 4:35 AM, Joshua Colp <span dir="ltr"><<a href="mailto:jcolp@digium.com" target="_blank">jcolp@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Oct 16, 2017, at 07:22 AM, Alexander Traud wrote:<br>
> Issue 1:<br>
> When I<br>
> 1) git review --draft ... on the command line of my computer,<br>
> 2) cherry pick that change to other branches via the Web interface, and<br>
> 3) publish those changes via the Web interface,<br>
> I have to reply with "recheck" on the initial review, the one I submitted<br>
> as draft via the command line, always. Could somebody look into that, so<br>
> the check is done automatically when a review is published = leaves the<br>
> draft mode? Or is there any other trick?<br></span></blockquote><div><br></div><div>Yeah I keep forgetting about that.  I'll get that fixed today.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
</span>The configuration for this is actually itself kept in Gerrit[1] so<br>
anyone can look and make changes. George would probably be the one who<br>
could most quickly do such a thing.<br>
<span class=""><br>
> Issue 2:<br>
> When I cherry pick a draft via the Web interface, it is public visible<br>
> (published) right away. I would have expected the cherry pick to stay a<br>
> draft. Is that a configuration thing in Gerrit or an issue of the Web<br>
> interface of Gerrit?<br>
<br>
</span>It's the web interface of Gerrit. I haven't found an option to make the<br>
draft status preserved.<br>
<span class=""><br>
> Issue 3:<br>
> When I cherry pick via the Web interface, the branch is appended to the<br>
> topic automatically, for example ASTERISK-xxxxx gets ASTERISK-xxxxx-13. I<br>
> have to edit that manually. Is there any other trick instead of cherry<br>
> picking via the command line?<br>
<br>
</span>It's Gerrit doing it, and there's no option I've found to change it.<br>
Manually doing it is the only way to ensure it doesn't get appended.<br>
<span class=""><br>
> Issue 4:<br>
> When an identical change is cherry picked to several branches, reviewers<br>
> comment sometimes in branch 13 and sometimes in the branch master. Is it<br>
> possible to streamline this, and always use one specific branch for<br>
> example the oldest LTS or the master branch?<br>
<br>
</span>We could certainly ask people to do this but there's no enforcement<br>
possible of it.<br>
<span class=""><br>
> Issue 5:<br>
> When I edit a commit message via the Web interface, there is a (red) hair<br>
> line after 72 columns. However according to<br>
> <<a href="http://wiki.asterisk.org/wiki/display/AST/Commit+Messages" rel="noreferrer" target="_blank">http://wiki.asterisk.org/<wbr>wiki/display/AST/Commit+<wbr>Messages</a>>, wrapping<br>
> should happen after 80 columns. This is confusing. Could somebody<br>
> configure this in Gerrit? Or is this a static value which is not<br>
> changeable?<br>
<br>
</span>Gerrit is already set to 80 for both the subject length and max line<br>
length of the commit message itself. The web editor is fairly new, so it<br>
may have some sort of fixed size in it for warning.<br>
<span class=""><br>
> Issue 6:<br>
> As with the previous Reviewboard software, I see many changes submitted<br>
> to Gerrit without any issue in Jira. That is allowed, I know. However,<br>
> Jira allows me to link issues. This linking helps the original author to<br>
> be aware of missed corner cases or defects of his change. Because some<br>
> changes go through without any issue report, I do not know how to link<br>
> those in Jira so their authors learn. Is there any trick, I am missing?<br>
<br>
</span>Not really. Everything gets linked based on the topic of the change<br>
matching a JIRA issue number. Without that there's no link.<br>
<span class=""><br>
> Issue 7:<br>
> The guidelines for commit messages state: "end the first sentence (and<br>
> any subsequent sentences) with punctuation." Even Digium employees do not<br>
> do that. That guideline is still in effect, isn't it?<br>
<br>
</span>Technically yes but it's a comparatively small thing in the grand scheme<br>
of things so it doesn't see much enforcement.<br>
<br>
[1] <a href="https://gerrit.asterisk.org/#/admin/projects/infrastructure" rel="noreferrer" target="_blank">https://gerrit.asterisk.org/#/<wbr>admin/projects/infrastructure</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Joshua Colp<br>
Digium, Inc. | Senior Software Developer<br>
445 Jan Davis Drive NW - Huntsville, AL 35806 - US<br>
Check us out at: <a href="http://www.digium.com" rel="noreferrer" target="_blank">www.digium.com</a> & <a href="http://www.asterisk.org" rel="noreferrer" target="_blank">www.asterisk.org</a><br>
</font></span><div class="HOEnZb"><div class="h5"><br>
--<br>
______________________________<wbr>______________________________<wbr>_________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" rel="noreferrer" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" rel="noreferrer" target="_blank">http://lists.digium.com/<wbr>mailman/listinfo/asterisk-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span style="font-size:12.8px">George Joseph</span><br style="font-size:12.8px"><span style="font-size:12.8px">Digium, Inc. | Software Developer</span><span style="font-size:12.8px"><br>445 Jan Davis Drive NW - Huntsville, AL 35806 - US<br></span><span style="font-size:12.8px">Check us out at: </span><a href="http://www.digium.com/" rel="noreferrer" style="color:rgb(17,85,204);font-size:12.8px" target="_blank">www.digium.com</a><span style="font-size:12.8px"> & </span><a href="http://www.asterisk.org/" rel="noreferrer" style="color:rgb(17,85,204);font-size:12.8px" target="_blank">www.asterisk.org</a><br><div><br></div></div></div>
</div></div>