+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. >
