Skip to content

Add clang-tidy configuration and CI job #47

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
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add clang-tidy configuration and CI job #47

wants to merge 5 commits into from

Conversation

unterumarmung
Copy link
Owner

Summary

  • configure clang-tidy to run core analyzer, diagnostic, bugprone and performance checks
  • treat Catch2 headers as system and disable POSIX signal handling to silence external warnings
  • run clang-tidy in CI against project headers only

Testing

  • cmake --build build
  • clang-tidy --format-style=file --warnings-as-errors=* -p build include/fixed_string.hpp

https://chatgpt.com/codex/tasks/task_e_68a83dd12a38832088936c3099c8315e

unterumarmung
unterumarmung commented Aug 22, 2025
View reviewed changes
unterumarmung
unterumarmung commented Aug 22, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
codex
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@unterumarmung