-
-
Notifications
You must be signed in to change notification settings - Fork 649
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Static Analysis results of doctest #83
Comments
Thanks for these! Currently doctest is per-push statically analyzed with these: Cppcheck / Clang-Tidy / Coverity Scan / OCLint / Visual Studio Analyzer (as can be seen here) - but to be clean I pass quite a bunch of suppression flags though... I'll look into the lists you supplied in a few weeks when I have more time - they are exactly the ones I'm not currently using. |
You're welcome. I had not looked at One on the most intriguing messages:
I like the order |
weird - I think short can be unsigned... I'm going through the logs at the moment and fixing some things, but not all - many of them I would suppress. |
martinmoene/doctest Static Analyses.
Contains
The text was updated successfully, but these errors were encountered: