Build with warnings as errors (#289)

MiguelCompany · richiware · web-flow · commit 9beec7e6934b · 2025-10-31T10:14:23.000+01:00
* Refs #23858. Build with warnings as errors.

Signed-off-by: Miguel Company &lt;miguelcompany@eprosima.com&gt;

* Refs #23858. Add option for MSVC

Signed-off-by: Miguel Company &lt;miguelcompany@eprosima.com&gt;

* Refs #23858. Fix warning.

Signed-off-by: Miguel Company &lt;miguelcompany@eprosima.com&gt;

* Refs #23858. Move to CI

Signed-off-by: Ricardo González Moreno &lt;ricardo@richiware.dev&gt;

* Apply suggestion

Co-authored-by: Miguel Company &lt;miguelcompany@eprosima.com&gt;
Signed-off-by: Ricardo González &lt;correoricky@gmail.com&gt;

---------

Signed-off-by: Miguel Company &lt;miguelcompany@eprosima.com&gt;
Signed-off-by: Ricardo González Moreno &lt;ricardo@richiware.dev&gt;
Signed-off-by: Ricardo González &lt;correoricky@gmail.com&gt;
Co-authored-by: Ricardo González Moreno &lt;ricardo@richiware.dev&gt;
diff --git a/.github/workflows/reusable-ci.yml b/.github/workflows/reusable-ci.yml
@@ -53,6 +53,7 @@ on:
         type: boolean
         default: false
 env:
+    colcon-build-default-cmake-args: ${{ contains(inputs.os-version, 'windows') && '-DCMAKE_CXX_FLAGS_INIT="/WX"' || '-DCMAKE_CXX_FLAGS_INIT="-Werror"' }}
     toolset: ${{ inputs.vs-toolset && format('-T {0}', inputs.vs-toolset) || '' }}
 defaults:
   run:
@@ -122,7 +123,7 @@ jobs:
         colcon_build_args: ${{ inputs.colcon-args }}
         colcon_build_args_default: --event-handlers=console_direct+
         cmake_args: ${{ inputs.cmake-args }}
-        cmake_args_default: ${{ env.toolset }}
+        cmake_args_default: ${{ env.colcon-build-default-cmake-args }} ${{ env.toolset }}
         cmake_build_type: ${{ matrix.cmake-build-type }}
         workspace: ${{ github.workspace }}
 
diff --git a/test/cdr/SimpleTest.cpp b/test/cdr/SimpleTest.cpp
@@ -2265,7 +2265,7 @@ TEST(CDRTests, DeserializeIntoANonEmptyMapInXCDRv1)
     };
 
     std::map<uint16_t, std::string> initialized_map{
-        {1, "a"}
+        {static_cast<uint16_t>(1u), "a"}
     };
 
     FastBuffer cdr_buffer(buffer, 14);
