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

Added checking and applying of clang-format #3

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

keeranroth
Copy link
Contributor

Add a convenience script to allow checking of clang format on files in the repo. This adds a bit nicer error reporting on top of 'git clang-format', and works in CI where 'git clang-format' may not be available.

We also add the github workflow to make sure that clang format is applied.

Usage of the script in CI is:

tools/run-clang-format.py check

and to apply clang-format to all .c, .cpp, .h and .hpp files, use

tools/run-clang-format.py apply

Add a convenience script to allow checking of clang format on files in
the repo. This adds a bit nicer error reporting on top of
'git clang-format', and works in CI where 'git clang-format' may not be
available.

We also add the github workflow to make sure that clang format is
applied.

Usage of the script in CI is:

  tools/run-clang-format.py check

and to apply clang-format to all .c, .cpp, .h and .hpp files, use

  tools/run-clang-format.py apply
@PingTakPeterTang PingTakPeterTang merged commit 49bc6da into main Dec 7, 2023
4 checks passed
@keeranroth keeranroth deleted the dev/keeranr/clang-format-check branch December 13, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants