<div dir="ltr"><div><div>I have raised a bug[1] to block these types of commits in the future. This is an unnecessary risk that we are taking.<br><br></div>I also think that we need to remove this as acceptable practice from the MDN page. <br><br></div>David<br><div><div><br>[1] <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1336459">https://bugzilla.mozilla.org/show_bug.cgi?id=1336459</a><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 3 February 2017 at 15:11, Ryan VanderMeulen <span dir="ltr"><<a href="mailto:rvandermeulen@mozilla.com" target="_blank">rvandermeulen@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>A friendly reminder that per the MDN commit rules, the use of "No bug" in the commit message is to be used sparingly - in general for minor things like whitespace changes/comment fixes/etc where traceability isn't as important.<br><a href="https://developer.mozilla.org/docs/Mozilla/Developer_guide/Committing_Rules_and_Responsibilities" target="_blank">https://developer.mozilla.org/<wbr>docs/Mozilla/Developer_guide/<wbr>Committing_Rules_and_<wbr>Responsibilities</a><br><br></div>I've come across uses of "No bug" commits recently where entire upstream imports of vendored libraries were done. This is bad for multiple reasons:<br></div>* If makes sheriffing difficult - if the patch is broken and needs backing out, where should the comment go? When it gets merged to mozilla-central, who gets notified?<br></div>* It makes tracking a pain - what happens if that patch ends up needing uplift? What about when that patch causes conflicts with another patch needing uplift? What if it causes a regression and a blocking bug needs to be filed?<br></div><div><br></div><div>Bugs are cheap. Please use them.<br><br></div><div>Thanks,<br></div><div>Ryan<br></div></div>
<br>______________________________<wbr>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
<br></blockquote></div><br></div>