C++ code has been committed without being static analyzed

[platisd]

trafficstars

Description

As an INS developer I do not want to have code that has not been approved by a static analysis tool committed to one of the main branches. Therefore, I would like to make sure that my code is verified by a static analysis tool before being allowed to merge.

A good relevant post to read can be found here.

Definition of Done

• A static analysis tool (e.g. cppcheck) is run by Travis and verifies the submitted code before allowing to merge
• Static analysis script is present in the c++ source folders, e.g. server/run_static_analysis.sh
• Current problems, discovered by static analysis tool, are addressed
• (Optional) Try to run the static analysis check in parallel with the rest of the build

[samup4web]

Clang-tidy :+1:

[platisd]

Up to the one who implements this! :smiley:

[platisd]

@samup4web are you currently working on this? If so please move this to the appropriate column in the board otherwise unassign yourself so maybe new members can take it! 😄

[samup4web]

Yes working on this, forgot to move it status.

[lexious89]

Is this issue solved? @samup4web