Skip to content

Commit

Permalink
Fix CMake CI and update workflow triggers (#3073)
Browse files Browse the repository at this point in the history
* Fix CMake CI and update workflow triggers

* formatting
  • Loading branch information
thomas-bc authored Dec 11, 2024
1 parent 05b817f commit e957d40
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 7 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/cmake-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,12 @@ on:
pull_request:
# The branches below must be a subset of the branches above
branches: [ devel, release/** ]
# Only run this test when CMake files and setup change
paths:
- 'cmake/**'
- '**/CMakeLists.txt'
- '**.cmake'
- 'requirements.txt'
- ".github/workflows/cmake-test.yml"
paths-ignore:
- 'docs/**'
- '**.md'
- '.github/actions/spelling/**'
- '.github/ISSUE_TEMPLATE/**'


# A workflow run is made up of one or more jobs that can run sequentially or in parallel
jobs:
Expand Down
3 changes: 3 additions & 0 deletions cmake/test/src/test_ref_shared.py
Original file line number Diff line number Diff line change
Expand Up @@ -141,6 +141,9 @@ def test_ref_module_info(REF_BUILD):
"SignalSetArrayAc.hpp",
"SignalTypeEnumAc.cpp",
"SignalTypeEnumAc.hpp",
"SignalGen_DpReqTypeEnumAc.cpp",
"SignalGen_DpReqTypeEnumAc.hpp",
"SignalGen_DpReqTypeEnumAi.xml",
]
actual_gen = [Path(source).name for source in generated]
assert sorted(expected_gen) == sorted(
Expand Down
3 changes: 3 additions & 0 deletions cmake/test/src/test_unittests.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,6 +167,9 @@ def test_unittest_module_ut_info(UT_BUILD):
"SignalGenTesterBase.cpp",
"SignalGenTesterBase.hpp",
"SignalGenTesterHelpers.cpp",
"SignalGen_DpReqTypeEnumAc.cpp",
"SignalGen_DpReqTypeEnumAc.hpp",
"SignalGen_DpReqTypeEnumAi.xml",
]
actual_gen = [Path(source).name for source in generated]
assert sorted(expected_gen) == sorted(
Expand Down

0 comments on commit e957d40

Please sign in to comment.