<div dir="ltr"><div><div>How does a commit-hook align with other discussions about enforcing code style? [Which I haven't followed closely, hence me asking!]<br><br></div>Biggest downside of a hook would seem that it's a requirement that's effectively hidden until one actually goes to commit. Seems like this would be something better to live as a mach command or script in the tree, until there are better ways to expose it as part of code review. (Yes, reviewers would need to remember to ask that the tool be run, but given the low impact of forgetting it doesn't seem like a big deal.)<br><br></div>Justin<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 28, 2015 at 12:22 PM, Tim Nguyen <span dir="ltr"><<a href="mailto:ntim.bugs@gmail.com" target="_blank">ntim.bugs@gmail.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>Hello Firefox Devs,<br></div>I have written an MDN doc [0] about cleaning up SVGs based on an older Etherpad [1]. I'm seeking feedback for this doc, so please tell me if there's something I should add/change/remove. <br><br></div>Also, Gijs suggested a commit hook, I haven't wrote one yet, but I've started a script [2], so once it's done, it should be easy to write a commit hook from there. If anyone could help me finish this script and set up the hook, it would be awesome too.<br><br></div>Thanks,<br></div>Tim Nguyen :ntim<br><div><div><div><div><br>[0] : <a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/SVG_Guidelines" target="_blank">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/SVG_Guidelines</a><br>[1] : <a href="https://etherpad.mozilla.org/svg-guidelines" target="_blank">https://etherpad.mozilla.org/svg-guidelines</a><br>[2] : <a href="https://gist.github.com/nt1m/21217e3e2ea5b0c0586b" target="_blank">https://gist.github.com/nt1m/21217e3e2ea5b0c0586b</a><br></div></div></div></div></div>
<br>_______________________________________________<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/listinfo/firefox-dev</a><br>
<br></blockquote></div><br></div>