-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add clang-tidy checks #263
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #263 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 5 5
Lines 455 455
=========================================
Hits 455 455 ☔ View full report in Codecov by Sentry. |
Will this apply fixes and recommit? Presumably, it will just fail and we need to fix it manually? If so perhaps we should also add the pre-commit bindings? |
This needs a newer version of CMake. Will pick this up again later. It's a distraction for now. |
Just pre-commit is probably more useful. I'll set a local helper script up
first
…On Sun, 19 Nov 2023 at 13:25, Antony Peacock ***@***.***> wrote:
Will this apply fixes and recommit? Presumably, it will just fail and we
need to fix it manually? If so perhaps we should also add the pre-commit
bindings?
—
Reply to this email directly, view it on GitHub
<#263 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAF5ZE4Q5XNJ42QM6R2ZDRDYFICEXAVCNFSM6AAAAAA7RXUJXGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXHA2TKMRRGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
No description provided.