+1 (binding) On Mon, Jan 15, 2018 at 9:43 AM, Marco de Abreu < [email protected]> wrote:
> +1 > > On Mon, Jan 15, 2018 at 6:27 PM, Pedro Larroy < > [email protected]> > wrote: > > > Hi > > > > I would like to propose to compile in CI with warnings as errors for > > increased code quality. This has a dual purpose: > > > > 1. Enforce a clean compilation output. Warnings often indicate > > deficiencies in the code and hide new warnings which can be an > > indicator of problems. > > > > 2. Warnings can surface bugs as has happened before. > > > > While this might be impractical in all architectures, I would propose > > having the Linux and Clang build run without warnings in CI. > > > > I think we are very close to this as I personally have been fixing > > warnings in Linux and OSX / Clang. > > > > References: > > > > https://github.com/apache/incubator-mxnet/pull/9398 > > > > http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/ > > incubator-mxnet/detail/PR-9398/1/pipeline > > > > Pedro. > > >
