Skip to content

Commit c9587e2

Browse files
committed
Let the CI fail if no wrappers are generated
- commented out for failing tasks, see #276 and #291
1 parent e32ca10 commit c9587e2

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

.github/workflows/build.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ jobs:
9797
with:
9898
name: wrappers_ubuntu_${{ matrix.container_version }}
9999
path: generated_cpp
100+
if-no-files-found: error
100101

101102
oldschool:
102103
strategy:
@@ -157,6 +158,8 @@ jobs:
157158
with:
158159
name: wrappers_${{ matrix.container_os }}-${{ matrix.container_os_version }}_${{ matrix.configuration }}
159160
path: generated_cpp
161+
# comment in after #291 is fixed
162+
# if-no-files-found: error
160163

161164
macOS:
162165
strategy:
@@ -243,10 +246,13 @@ jobs:
243246
QTDIR=-UNDEFINED- ./pythonqt_generator --qt-version=${{ steps.versions.outputs.QT_VERSION_FULL }} --include-paths=$QT_ROOT_DIR/lib
244247
245248
- name: Upload Wrappers
249+
if: ${{ contains(matrix.configuration, 'release') }}
246250
uses: actions/upload-artifact@v4
247251
with:
248252
name: wrappers_macos${{ steps.versions.outputs.MACOS_VERSION_SHORT }}_qt${{ steps.versions.outputs.QT_VERSION_SHORT }}
249253
path: generated_cpp
254+
# comment in after #276 is fixed
255+
# if-no-files-found: error
250256

251257
windows:
252258
strategy:
@@ -358,3 +364,4 @@ jobs:
358364
with:
359365
name: wrappers_${{ matrix.qt-arch }}_${{ steps.versions.outputs.QT_VERSION_SHORT }}
360366
path: generated_cpp
367+
if-no-files-found: error

.github/workflows/build_latest.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ jobs:
9595
with:
9696
name: wrappers_${{ matrix.os }}_${{ steps.setenv.outputs.QT_VERSION_SHORT }}
9797
path: generated_cpp
98+
if-no-files-found: error
9899

99100
- name: Build PythonQt Ubuntu
100101
if: ${{ matrix.os == 'ubuntu' }}

0 commit comments

Comments
 (0)