Skip to content

CI hygiene: pin numsim-cas, enable -Werror on tests, add UBSan/ASan job #13

Description

@petlenz

Summary

Three related CI gaps:

1. `numsim-cas` pinned to a moving branch

`CMakeLists.txt:43` sets `GIT_TAG main` for the `FetchContent` of `numsim-cas`. A breaking change upstream will flip CI red without any commit here, and an old green build is unreproducible.

Fix: pin to the current `numsim-cas` HEAD SHA. Bump deliberately when needed.

2. No -Werror / no warning flags on tests

`numsim_codegen` is an INTERFACE target so it can't carry compile options directly, but the test target can:

target_compile_options(numsim_codegen_tests PRIVATE -Wall -Wextra -Wpedantic -Werror)

This locks down the generator, driver, and all unit tests.

3. No sanitizer matrix entry

A clang+Debug job with `-fsanitize=address,undefined -fno-sanitize-recover=all` would catch:

Add a matrix entry in `.github/workflows/build.yml`.

Where

  • `CMakeLists.txt:40-46`
  • `.github/workflows/build.yml`
  • `tests/CMakeLists.txt`

Refs

  • REVIEW.md → CI-A1 (High), CORR-17, CORR-18, CORR-19, CORR-20
  • Severity: High

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions