<div dir="ltr">On Wed, Jun 6, 2018 at 1:12 PM Loïc Hoguin <<a href="mailto:essen@ninenines.eu">essen@ninenines.eu</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It's worth noting that you did not have an issue with Cowboy, Ranch and <br>
friends. That's because the default policy in Erlang.mk is to NOT use <br>
warnings_as_errors for dependencies (and as a result this option is not <br>
included in the generated rebar.config).<br>
<br></blockquote></div><div class="gmail_quote"><br></div><div class="gmail_quote">IIRC, Rebar doesn't include the option by default either. But people like to add that option as a way to rid themselves of warnings in their projects. Normally, this is a fine and sensible thing to do, but I'm currently a bit torn on its usefulness in libraries other people rely on, and I might want to recommend people not adding it to those.<br></div><div><br></div><div>Generally I think we agree here.<br></div><div><br></div><div>Individual projects can override this setting when you assemble the software on a grander scale and force warnings_as_errors. It is always easier to add strictness at a higher level. But even then: if you don't control all of the source code, you must be prepared for parts of it lagging behind in warning count and/or quality.</div><div><br></div></div>