Skip to content

Commit

Permalink
Refs #22174: Add Debug CI to master nightly job
Browse files Browse the repository at this point in the history
Signed-off-by: eProsima <jesuspoderoso@eprosima.com>
  • Loading branch information
JesusPoderoso committed Nov 14, 2024
1 parent 44310c4 commit 944b16d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/nightly-windows-master.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,13 @@ jobs:
security:
- 'ON'
- 'OFF'
cmake_build_type:
- 'Debug'
- 'RelWithDebInfo'
uses: eProsima/Fast-DDS/.github/workflows/reusable-windows-ci.yml@master
with:
label: 'nightly-sec-${{ matrix.security }}-windows-ci-master'
cmake-config: ${{ matrix.cmake_build_type }}
cmake-args: "-DSECURITY=${{ matrix.security }}"
ctest-args: "-LE xfail"
fastdds_branch: 'master'
9 changes: 6 additions & 3 deletions .github/workflows/reusable-windows-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,11 @@ on:
description: 'Branch or tag of Fast DDS repository (https://github.com/eProsima/Fast-DDS)'
required: true
type: string
cmake-config:
description: 'CMake configuration to use'
required: false
type: string
default: 'RelWithDebInfo'

defaults:
run:
Expand All @@ -34,8 +39,6 @@ jobs:
strategy:
fail-fast: false
matrix:
cmake-config:
- 'RelWithDebInfo'
vs-toolset:
- 'v142'
steps:
Expand Down Expand Up @@ -142,7 +145,7 @@ jobs:
# The following Fast DDS CMake options need to be specified here instead of in the meta files
# because they vary from platform to platform
cmake_args_default: ${{ inputs.cmake-args }} -T ${{ matrix.vs-toolset }} -DTHIRDPARTY_Asio=FORCE -DTHIRDPARTY_TinyXML2=FORCE -DTHIRDPARTY_fastcdr=OFF -DTHIRDPARTY_UPDATE=ON -DEPROSIMA_EXTRA_CMAKE_CXX_FLAGS="/MP /WX"
cmake_build_type: ${{ matrix.cmake-config }}
cmake_build_type: ${{ inputs.cmake-config }}
workspace: ${{ github.workspace }}

- name: Test
Expand Down

0 comments on commit 944b16d

Please sign in to comment.