PengZheng commented on PR #817: URL: https://github.com/apache/celix/pull/817#issuecomment-3853360988
>Maybe already good to minimise the usage of the gcc static analyzer to a single build? I think, not sure, that there is no benefit to run the gcc static analyzer in the debug build (compared to the RelWithDebInfo build) and seeing the latest build I expect the analyzer performs better with a RelWithDebInfo build. +1 Moreover, it does not make sense to enable analyzer for tests. Note that half of our code base is for testing code. I suggest that static analyzer should only be enabled in the [Create Celix Step](https://github.com/apache/celix/blob/effe33a1ae3ad8f7782e301b7c5a69b48e1e9f9c/.github/workflows/conan_create.yml#L69) in `conan_create.yml`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
