Skip to content
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

Tests: Add a CMake option to disable the main catch all exceptions #2076

Merged
merged 2 commits into from
Mar 17, 2025
Merged

Tests: Add a CMake option to disable the main catch all exceptions #2076

merged 2 commits into from
Mar 17, 2025

Conversation

mwestphal
Copy link
Contributor

@mwestphal mwestphal commented Mar 16, 2025 •
edited
Loading

  • Add a CMake option to control the behavior of the main.cxx catch all exceptions
  • Use it in CI
  • Coverage reduction is expected and I believe a good idea to unsure this part of the code is not covered.

Needs: #2065

@mwestphal mwestphal requested a review from Meakk March 17, 2025 09:35
@Meakk
Copy link
Member

Meakk commented Mar 17, 2025

I don't get what's the point of this change.

@mwestphal
Copy link
Contributor Author

I don't get what's the point of this change.

Discussed on discord: https://discord.com/channels/1046005690809978911/1203365728619602001/1351143579006795849

@mwestphal mwestphal merged commit a33113c into f3d-app:master Mar 17, 2025
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Meakk Meakk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mwestphal @Meakk