search

OpenCyphal / libcanard

A compact implementation of the Cyphal/CAN protocol in C for high-integrity real-time embedded systems
http://opencyphal.org
MIT License
327 stars 192 forks source link

Remove SonarCloud cache and threads setup as it is now offered by default #208

Closed mpaladin closed 1 year ago

mpaladin commented 1 year ago

No need to configure the cache and threads anymore, SonarCloud now has an automatic analysis caching and threads detection. See https://docs.sonarcloud.io/advanced-setup/languages/c-c-objective-c/#analysis-cache.

pavel-kirienko commented 1 year ago

I understand that the cache is enabled by default, which is not what we want, hence the explicit override. Caching was known to cause the analysis to fail.

mpaladin commented 1 year ago

I understand that the cache is enabled by default, which is not what we want, hence the explicit override. Caching was known to cause the analysis to fail.

@pavel-kirienko thank you for your prompt answer. What failure did you face? I couldn't find the note on the git blame. Wouldn't you be willing to give it a try again? Your analysis would have a good speed up. And, more importantly, if the analysis fails, I would be happy to investigate it as we currently don't have any open bugs.