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.

Reply via email to