Open-Agriculture / AgIsoStack-plus-plus

AgIsoStack++ is the completely free open-source C++ ISOBUS library for everyone
https://agisostack.com/
MIT License
187 stars 41 forks source link

Switch to debug builds in CI to catch asserts in unit tests #333

Closed GwnDaan closed 12 months ago

GwnDaan commented 12 months ago

Tests were failing in the CI pipeline because it couldn't catch any asserts as it was being build in "Release" mode, which optimizes out the asserts. I think it's reasonable to build the pipeline in debug to pass the unit tests, opinions?

sonarcloud[bot] commented 12 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

GwnDaan commented 12 months ago

maybe it would be nice to have some on-tag action that could compile both the release and debug versions of the library?

Yeah, that would be nice to have once we have our first real release