
<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Would it be better if 'CLA: trivial’ is in the commit message but no CLA on file, then a new label like ’warn: no CLA but trivial’ is added? This can inform the committer who will merge the PR for the CLA condition of the commits.<div class=""><br class=""></div><div class=""><div class="">

<div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;"><br class="Apple-interchange-newline">Regards,</div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;"><br class=""></div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;">Paul Yang</div>

</div>

<div><br class=""><blockquote type="cite" class=""><div class="">On Dec 12, 2019, at 5:29 PM, Dmitry Belyavsky <<a href="mailto:beldmit@gmail.com" class="">beldmit@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Dear Matt,<div class=""><br class=""></div><div class="">As </div><div class="">- the contributor agreed to sign the CLA and </div><div class="">- there was a mark that CLA is signed and</div><div class="">- all the necessary approves were present</div><div class="">I decided that there is no problem to merge.</div><div class=""><br class=""></div><div class="">BTW, I am not sure the PR was trivial enough.</div><div class=""><br class=""></div><div class="">Anyway, the responsibility was mine, not the git one :)</div><div class=""><br class=""></div></div><br class=""><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 12, 2019 at 12:20 PM Matt Caswell <<a href="mailto:matt@openssl.org" class="">matt@openssl.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I notice that PR 10594 (Add support for otherName:NAIRealm in output)<br class="">

got merged yesterday:<br class="">

<a href="https://github.com/openssl/openssl/pull/10594" rel="noreferrer" target="_blank" class="">https://github.com/openssl/openssl/pull/10594</a><br class="">

<br class="">

The commit description contained "CLA: trivial" and so the "hold: cla<br class="">

required" label was not automatically applied to the PR. But the<br class="">

discussion in the PR suggested a CLA should be submitted. But it got<br class="">

merged anyway! Fortunately the CLA had already been processed - just not<br class="">

noted in the PR. So, in this case, it makes no difference.<br class="">

<br class="">

I think this points to a possible flaw in our workflow for dealing with<br class="">

trivial changes. Because the "CLA: trivial" header suppresses the "hold:<br class="">

cla required" label and the git hooks don't complain when commits get<br class="">

pushed with the "CLA: trivial" header and no CLA on file, it seems<br class="">

possible to me that we could push commit all the way through the process<br class="">

without the reviewers even realising that the author is claiming<br class="">

triviality on the commit.<br class="">

<br class="">

Not sure what the solution to that is.<br class="">

<br class="">

Matt<br class="">

</blockquote></div><br clear="all" class=""><div class=""><br class=""></div>-- <br class=""><div dir="ltr" class="gmail_signature">SY, Dmitry Belyavsky</div>

</div></blockquote></div><br class=""></div></body></html>