-
Notifications
You must be signed in to change notification settings - Fork 70
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 documentation for running clang-format #60
Conversation
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.
Thanks @sherry-yuan, your documentation will make it much easier to pass the clang-format
workflow on first try.
1bd97df
to
a01ecd8
Compare
Thanks @sherry-yuan. I cosmetically revised the Markdown to make it somewhat readable in raw format, but otherwise left everything as is. Good thinking on writing a temporary test of a failing |
Since I noticed you committed additions with message
|
Document which clang version to run to avoid workflow failure, clang-format git integration as well installation instruction.
a01ecd8
to
35aa2e6
Compare
I see, thanks for the tip @pcolberg ! |
Document which clang version to run to avoid workflow failure, clang-format git integration as well installation instruction.