-
Notifications
You must be signed in to change notification settings - Fork 12
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-format configuration and format sources #420
Conversation
798fdcd
to
c209f5e
Compare
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #420 +/- ##
==========================================
- Coverage 69.37% 69.33% -0.04%
==========================================
Files 11 11
Lines 875 874 -1
==========================================
- Hits 607 606 -1
Misses 268 268
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
99.9% OK for me.
It will never be perfect in one go, there are still some places where I prefer the old formatting but overall it's way better than before
Add `.git-blame-ignore-revs`, with the previous code formatting commit. It allows git blame tools to ignore the specified commits. Command to configure it: `git config blame.ignoreRevsFile .git-blame-ignore-revs`
c209f5e
to
9eac339
Compare
Description
.git-blame-ignore-revs
to ignore formatting commit when blamingChanges include