search

crashappsec / libcon4m

Base Compiler and Runtime Support for con4m
Apache License 2.0
0 stars 0 forks source link

enable tests in ci #34

Closed indecisivedragon closed 3 months ago

ee7 commented 3 months ago

~The tests aren't in a place to enable in CI yet~. For example, we don't have a command for which an exit status of 0 implies that the tests pass.

ee7 commented 3 months ago

The tests aren't in a place to enable in CI yet.

I wrote that, but after https://github.com/crashappsec/libcon4m/issues/37 I think this is already well worth doing, even if the workflow indicating success does not currently imply that the tests pass.

I created https://github.com/crashappsec/libcon4m/pull/38 as something to iterate on. Let's say that, if merged, that PR can close this ticket; we can open follow-up tickets for further improvements.