Skip to content

Commit 73236f5

Browse files
complete warnings as errors
1 parent ac3b57b commit 73236f5

2 files changed

Lines changed: 89 additions & 12 deletions

File tree

.github/workflows/ci.yml

Lines changed: 33 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,15 @@ env:
1515

1616
jobs:
1717
linux-release:
18-
name: Linux Release (FluxGraph OFF)
18+
name: ${{ matrix.lane }}
1919
runs-on: ubuntu-24.04
20+
strategy:
21+
fail-fast: false
22+
matrix:
23+
include:
24+
- lane: Linux Release (FluxGraph OFF, strict)
25+
configure_preset: ci-linux-release-strict
26+
test_preset: ci-linux-release-strict
2027
steps:
2128
- name: Checkout
2229
uses: actions/checkout@v4
@@ -61,15 +68,22 @@ jobs:
6168

6269
- name: Build
6370
run: |
64-
cmake --preset ci-linux-release
65-
cmake --build --preset ci-linux-release --parallel
71+
cmake --preset "${{ matrix.configure_preset }}"
72+
cmake --build --preset "${{ matrix.configure_preset }}" --parallel
6673
6774
- name: Test (CTest provider baseline)
68-
run: ctest --preset ci-linux-release -L provider
75+
run: ctest --preset "${{ matrix.test_preset }}" -L provider
6976

7077
windows-release:
71-
name: Windows Release (FluxGraph OFF)
78+
name: ${{ matrix.lane }}
7279
runs-on: windows-2022
80+
strategy:
81+
fail-fast: false
82+
matrix:
83+
include:
84+
- lane: Windows Release (FluxGraph OFF, strict)
85+
configure_preset: ci-windows-release-strict
86+
test_preset: ci-windows-release-strict
7387
steps:
7488
- name: Checkout
7589
uses: actions/checkout@v4
@@ -111,16 +125,23 @@ jobs:
111125
- name: Build
112126
shell: pwsh
113127
run: |
114-
cmake --preset ci-windows-release
115-
cmake --build --preset ci-windows-release --parallel
128+
cmake --preset "${{ matrix.configure_preset }}"
129+
cmake --build --preset "${{ matrix.configure_preset }}" --parallel
116130
117131
- name: Test (CTest provider baseline)
118132
shell: pwsh
119-
run: ctest --preset ci-windows-release -L provider
133+
run: ctest --preset "${{ matrix.test_preset }}" -L provider
120134

121135
linux-fluxgraph:
122-
name: Linux Release (FluxGraph ON)
136+
name: ${{ matrix.lane }}
123137
runs-on: ubuntu-24.04
138+
strategy:
139+
fail-fast: false
140+
matrix:
141+
include:
142+
- lane: Linux Release (FluxGraph ON, strict)
143+
configure_preset: ci-linux-release-fluxgraph-strict
144+
test_preset: ci-linux-release-fluxgraph-strict
124145
steps:
125146
- name: Checkout provider-sim
126147
uses: actions/checkout@v4
@@ -192,8 +213,8 @@ jobs:
192213
193214
- name: Build provider-sim (FluxGraph ON)
194215
run: |
195-
cmake --preset ci-linux-release-fluxgraph -DFLUXGRAPH_DIR="$GITHUB_WORKSPACE/fluxgraph"
196-
cmake --build --preset ci-linux-release-fluxgraph --parallel
216+
cmake --preset "${{ matrix.configure_preset }}" -DFLUXGRAPH_DIR="$GITHUB_WORKSPACE/fluxgraph"
217+
cmake --build --preset "${{ matrix.configure_preset }}" --parallel
197218
198219
- name: Test FluxGraph suites
199220
run: |
@@ -202,4 +223,4 @@ jobs:
202223
echo "ERROR: fluxgraph-server binary not found under $GITHUB_WORKSPACE/fluxgraph/build-server" >&2
203224
exit 1
204225
fi
205-
FLUXGRAPH_SERVER_EXE="$SERVER_BIN" ctest --preset ci-linux-release-fluxgraph -L fluxgraph
226+
FLUXGRAPH_SERVER_EXE="$SERVER_BIN" ctest --preset "${{ matrix.test_preset }}" -L fluxgraph

CMakePresets.json

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,12 +86,33 @@
8686
"CMAKE_BUILD_TYPE": "Release"
8787
}
8888
},
89+
{
90+
"name": "ci-linux-release-strict",
91+
"inherits": "ci-linux-release",
92+
"cacheVariables": {
93+
"ANOLIS_PROVIDER_SIM_WARNINGS_AS_ERRORS": "ON"
94+
}
95+
},
96+
{
97+
"name": "ci-windows-release-strict",
98+
"inherits": "ci-windows-release",
99+
"cacheVariables": {
100+
"ANOLIS_PROVIDER_SIM_WARNINGS_AS_ERRORS": "ON"
101+
}
102+
},
89103
{
90104
"name": "ci-linux-release-fluxgraph",
91105
"inherits": "ci-linux-release",
92106
"cacheVariables": {
93107
"ENABLE_FLUXGRAPH": "ON"
94108
}
109+
},
110+
{
111+
"name": "ci-linux-release-fluxgraph-strict",
112+
"inherits": "ci-linux-release-fluxgraph",
113+
"cacheVariables": {
114+
"ANOLIS_PROVIDER_SIM_WARNINGS_AS_ERRORS": "ON"
115+
}
95116
}
96117
],
97118
"buildPresets": [
@@ -131,9 +152,22 @@
131152
"configurePreset": "ci-windows-release",
132153
"configuration": "Release"
133154
},
155+
{
156+
"name": "ci-linux-release-strict",
157+
"configurePreset": "ci-linux-release-strict"
158+
},
159+
{
160+
"name": "ci-windows-release-strict",
161+
"configurePreset": "ci-windows-release-strict",
162+
"configuration": "Release"
163+
},
134164
{
135165
"name": "ci-linux-release-fluxgraph",
136166
"configurePreset": "ci-linux-release-fluxgraph"
167+
},
168+
{
169+
"name": "ci-linux-release-fluxgraph-strict",
170+
"configurePreset": "ci-linux-release-fluxgraph-strict"
137171
}
138172
],
139173
"testPresets": [
@@ -197,12 +231,34 @@
197231
"outputOnFailure": true
198232
}
199233
},
234+
{
235+
"name": "ci-linux-release-strict",
236+
"configurePreset": "ci-linux-release-strict",
237+
"output": {
238+
"outputOnFailure": true
239+
}
240+
},
241+
{
242+
"name": "ci-windows-release-strict",
243+
"configurePreset": "ci-windows-release-strict",
244+
"configuration": "Release",
245+
"output": {
246+
"outputOnFailure": true
247+
}
248+
},
200249
{
201250
"name": "ci-linux-release-fluxgraph",
202251
"configurePreset": "ci-linux-release-fluxgraph",
203252
"output": {
204253
"outputOnFailure": true
205254
}
255+
},
256+
{
257+
"name": "ci-linux-release-fluxgraph-strict",
258+
"configurePreset": "ci-linux-release-fluxgraph-strict",
259+
"output": {
260+
"outputOnFailure": true
261+
}
206262
}
207263
]
208264
}

0 commit comments

Comments
 (0)