Skip to content

Let the CI fail if no wrappers are generated #282

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 1, 2025
Merged

Let the CI fail if no wrappers are generated #282

merged 1 commit into from
Sep 1, 2025

Conversation

@mrbean-bremen
Copy link
Contributor

usiems
usiems approved these changes Aug 29, 2025
View reviewed changes
Copy link
Contributor

@usiems usiems left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not familiar with the CI syntax, but this looks ok to me.

I think we should wait for the MacOS binding generation being fixed before merging this, otherwise all other merge requests will seem to cause new failures.

@mrbean-bremen
Copy link
Contributor Author

As discussed: commented out for the failing tasks, shall be re-added after #276 and #291 are fixed.

@mrbean-bremen
Let the CI fail if no wrappers are generated
c9587e2 
- commented out for failing tasks, see #276 and #291
@mrbean-bremen mrbean-bremen merged commit d2c4505 into master Sep 1, 2025
19 checks passed
@mrbean-bremen mrbean-bremen deleted the wrapper-errors branch September 1, 2025 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@usiems usiems usiems approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mrbean-bremen @usiems