-
Notifications
You must be signed in to change notification settings - Fork 799
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
Cpp clangformat #853
Cpp clangformat #853
Conversation
(cherry picked from commit 0179a55c1ce20c449d8a60d3966e821552ecffdc)
# Conflicts: # sdks/cpp/CMakeLists.txt
Build Failed 😱 Build Id: c7bb3a0d-a521-4d60-be3a-280b322b93a3 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: 96f7f9b1-1eaf-4a1a-8336-2dc3dae67f26 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 6cf316f5-750e-461d-93e6-79e755ff974d The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Build Succeeded 👏 Build Id: a1516f0f-fe23-4cb6-898a-2cd4393c67ee The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Not ready yet... |
Support for clang-format code formatting and code style verification. Issue: #713
Note that this PR is based on #803
Added 2 new cmake targets:
clang-format-apply
- calls clang-format before building sources (only for non-generated files)clang-format-verify
- verify that current branch:commit follows google code style (a build step to verify code style of pull requests)