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.

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

Workflow for code analysis #1249

Closed

Conversation

simonsan
Copy link
Contributor

@simonsan simonsan commented May 2, 2020

Early stage, aims for:

  • giving feedback if bugs are found
  • optimize that patch-formatting
  • publish fixes (patches) for found errors based on settings in .clang-tidy
  • publish fixes (patches) based on settings in .clang-format
  • use it just on the recent pushed commit, not over all the code base
    • include switch, we want to check the code base and fix stuff from time to time
  • LINK_WHAT_YOU_USE
  • make use of clazy (Qt related compiler warnings)
  • cpp-lint (not needed => Clang-Format)

Buildsystem-Flags
Q: Does it make sense to create a diagnose module under buildsystem for make diagnose or would it be enough to document the following switches and create some CMake file around it or so for exporting fixes (incl. Clang-Tidy that is already existing)

Future:

ref.: #1196

@simonsan simonsan added code quality Does not alter behavior, but beauty of our code ci Related to our continuous integration systems (kevin, github-actions, ...) labels May 2, 2020
@simonsan simonsan force-pushed the workflows/create-analysis branch 2 times, most recently from 5fde299 to 1da389f Compare May 2, 2020 23:58
@simonsan simonsan changed the title [WIP] Workflow for code analysis (clang-tidy, cppcheck, scan-build, ...) [WIP] Workflow for code analysis May 3, 2020
…idy, clang-format, iwyu, code compliance, scan-build report
@simonsan simonsan force-pushed the workflows/create-analysis branch from 50c5653 to 150a0bb Compare May 3, 2020 04:54
@simonsan simonsan changed the title [WIP] Workflow for code analysis Workflow for code analysis May 9, 2020
@simonsan
Copy link
Contributor Author

(Outdated)

@simonsan simonsan closed this Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Related to our continuous integration systems (kevin, github-actions, ...) code quality Does not alter behavior, but beauty of our code
Projects
Development

Successfully merging this pull request may close these issues.

1 participant