celix icon indicating copy to clipboard operation
celix copied to clipboard

Update ubuntu workflow to also build RelWithDebInfo and execute the unit tests

Open rlenferink opened this issue 2 years ago • 2 comments

I think it would be good to also execute the RelWithDebInfo build and unit tests.

This reduces the possibility of issues being found during a release (context).

Suggestion: update the .github/workflows/ubuntu.yml to execute a matrix build for:

  • Conan
    • ~~Debug -> all features enabled~~
    • ~~RelWithDebInfo -> all features enabled~~
    • ~~RelWithDebInfo -> only the default features enabled~~
  • Apt
    • Debug -> all features enabled
    • RelWithDebInfo -> all features enabled
    • RelWithDebInfo -> only the default features enabled

rlenferink avatar Sep 24 '23 12:09 rlenferink

RelWithDebInfo -> all features enabled for Conan has been addressed by #657.

PengZheng avatar Sep 25 '23 03:09 PengZheng

I should reminds that for Conan build, all features are off by default.

PengZheng avatar Oct 07 '23 07:10 PengZheng