Skip to content

feat: 581 - allow to implement different Asserts functions. implement some more assert utility functions#590

Merged
rokath merged 6 commits intorokath:mainfrom
KammutierSpule:581
Dec 14, 2025
Merged

feat: 581 - allow to implement different Asserts functions. implement some more assert utility functions#590
rokath merged 6 commits intorokath:mainfrom
KammutierSpule:581

Conversation

@KammutierSpule
Copy link
Contributor

Suggestion fix for #581

@github-actions github-actions bot added tests AnyChange go Pull requests that update go code C If any *.c or *.h file is changed in typical C locations. labels Dec 12, 2025
@KammutierSpule
Copy link
Contributor Author

I suggest to keep this open and I will fix things on this same PR. So no need to close and open new PullRequests.
Unless you want to drop it at all

I didn't tested the source code, only using internal tests. I will do it now

@KammutierSpule
Copy link
Contributor Author

I didn't tested the source code, only using internal tests. I will do it now

I tested on a project, it builds

@rokath rokath merged commit 5b5493b into rokath:main Dec 14, 2025
5 of 7 checks passed
@rokath
Copy link
Owner

rokath commented Dec 14, 2025

Thanks, this looks good! 👍 You (probaly) forgot to remove #define triceAssertFail(...) ... in triceOff.h I will fix that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AnyChange C If any *.c or *.h file is changed in typical C locations. go Pull requests that update go code tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants