<div dir="ltr"><div dir="ltr">Thanks Darran, all valid suggestions. <div><br></div><div>I've moved the document to my personal public account [1] - it should be accessible to anyone now.</div><div><br></div><div>I will incorporate your suggestions in the gist.</div><div><br></div><div>[1] <a href="https://docs.google.com/document/d/13DScS-WyQipblfDG-hjFJ12mQ7OM18TsAY-vIo6qAr8/edit?usp=sharing">https://docs.google.com/document/d/13DScS-WyQipblfDG-hjFJ12mQ7OM18TsAY-vIo6qAr8/edit?usp=sharing</a><br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>Thank you,</div><div>Martin<br></div></div></div></div></div></div></div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 10, 2019 at 12:54 PM Darran Lofthouse <<a href="mailto:darran.lofthouse@jboss.com">darran.lofthouse@jboss.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">As an open source project may be better to document somewhere visible to the community.<div><br></div><div>From the initial rules I would suggest drop the "NO JIRA REQUIRED" from commit messages and PR titles, the commit messages and potentially the titles end up in the history of the project and have no meaning outside of the tool.</div><div><br></div><div>At the moment it looks like the template only covers WildFly so may need a second template to cover WildFly Core.</div><div><br></div><div>I think from the perspective of the tool making it mandatory for each commit in the PR to contain a Jira ID either with or without the brackets is fine - unless you want to guarantee whitespace around the Jira ID maybe the regular expression could be simplified to ignore the brackets.</div><div><br></div><div>Also I think it is reasonable for the tool to expect a link in each PR which should also relate to the project the PR is against.</div><div><br></div><div>Regards,</div><div>Darran Lofthouse.</div><div><br></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 10, 2019 at 11:35 AM Martin Stefanko <<a href="mailto:mstefank@redhat.com" target="_blank">mstefank@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">For the demo I've started with <a href="https://gist.github.com/xstefank/3a79f2f199ee1e449a44c607120a9d30" target="_blank">this</a>. For additional checks I have only something to verify that issue in title matches the link in the description. Any ideas are welcome!<div><br></div><div>I was also thinking about making distinction between issues / bugs and features if that would be useful.</div><div><br></div><div>I've created a google document to track this - <a href="https://docs.google.com/document/d/1ZuD5fyOlGTVDh96jiuM6jTdofm4fG6gZqdfBNRdZp8I/edit?usp=sharing" target="_blank">https://docs.google.com/document/d/1ZuD5fyOlGTVDh96jiuM6jTdofm4fG6gZqdfBNRdZp8I/edit?usp=sharing</a>.<br><div><div><div dir="ltr" class="gmail-m_-4363263254654823632gmail-m_-7057904775992588065gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>Martin<br></div></div></div></div></div></div></div><br></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 10, 2019 at 11:10 AM Darran Lofthouse <<a href="mailto:darran.lofthouse@jboss.com" target="_blank">darran.lofthouse@jboss.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Do we have somewhere yet that we can start to look into an initial set of rules to discuss what the initial set should be?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 10, 2019 at 6:32 AM Martin Stefanko <<a href="mailto:mstefank@redhat.com" target="_blank">mstefank@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">> The time the cross becomes a problem is if we adopt a set of verification rules that we know will regularly trigger a failure to be ignored, if the failure cases to be ignored are truly exceptional cases then the cross will be useful.<div><br></div><div>It would be beneficial to adjust rules as we'll see how useful they are as time progresses. Configuration of rules is external to the service so it will be possible.</div><div><div dir="ltr" class="gmail-m_-4363263254654823632gmail-m_-7057904775992588065gmail-m_-157675329574063867gmail-m_1576121774971108453gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>Martin <br></div></div></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 9, 2019 at 5:49 PM Darran Lofthouse <<a href="mailto:darran.lofthouse@jboss.com" target="_blank">darran.lofthouse@jboss.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-family:"trebuchet ms",sans-serif">> OTOH if the mergers decide they want to do something I don't want a tool preventing us doing it, which is what my questions were driving at. We're way smarter than the tool.</div><div style="font-family:"trebuchet ms",sans-serif"><br></div><div style="font-family:"trebuchet ms",sans-serif">The status should not prevent a possibility of merge but if the cross next to the commit would be a problem we can add an override to disable tyr check (meaning make it pass manually) with a comment or similar as suggested above.</div></div></blockquote><div><br></div><div>The time the cross becomes a problem is if we adopt a set of verification rules that we know will regularly trigger a failure to be ignored, if the failure cases to be ignored are truly exceptional cases then the cross will be useful.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div dir="ltr" class="gmail-m_-4363263254654823632gmail-m_-7057904775992588065gmail-m_-157675329574063867gmail-m_1576121774971108453gmail-m_39407116264145588gmail-m_8306954015641473860gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>Martin<br></div></div></div></div></div></div></div><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
</blockquote></div>
</blockquote></div></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>