Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add cppcheck & clang-tidy to CI #222

Closed
markos opened this issue Jan 11, 2024 · 2 comments
Closed

Add cppcheck & clang-tidy to CI #222

markos opened this issue Jan 11, 2024 · 2 comments
Assignees
@gtsoul-tech
Labels
enhancement New feature or request
Milestone
5.4.12

Comments

@markos
Copy link

markos commented Jan 11, 2024 •
edited
Loading

Add cppcheck and clang-tidy for each file in the build for static code analysis.
@markos markos added the enhancement New feature or request label Jan 11, 2024
@markos markos mentioned this issue Jan 11, 2024
Closed
@markos markos added this to the 5.4.12 milestone Jan 11, 2024
markos added a commit that referenced this issue Jan 20, 2024
@markos
Merge pull request #225 from VectorCamp/feature/cleanup-compiler-warn…
98eb459 
…ings

According to https://buildbot-ci.vectorcamp.gr/#/changes/93

most builds succceded and with no compiler warnings. The build failures were only on x86 and Arm for SIMDe builds: x86 because of a bug in SIMDe emulation of own x86 intrinsics in non-native mode and Arm due to clang, unsure if this is actually a bug in SIMDe or clang itself. All the remaining compiler warnings that were suppressed was because they were not possible to fix for the scope of this project. 

This PR will close #170, code quality improvements however will continue with the integration of #222 or similar static code analyzer to CI and continuous refactoring.
@markos markos mentioned this issue Jan 20, 2024
Merged
@markos markos changed the title Add cppcheck to Cmake Add cppcheck & clang-tidy to CI Apr 16, 2024
@markos
Copy link
Author

markos commented Apr 19, 2024

cppcheck added: https://buildbot-ci.vectorcamp.gr/#/builders/269/builds/2/steps/5/logs/stdio

@markos
Copy link
Author

markos commented Apr 22, 2024

clang-tidy added: https://buildbot-ci.vectorcamp.gr/#/builders/282/builds/2

@markos markos closed this as completed Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gtsoul-tech gtsoul-tech

Labels
enhancement New feature or request
Projects
None yet
Milestone
5.4.12
Development

No branches or pull requests
2 participants
@markos @gtsoul-tech