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

Update and enforce formatting for C/C++ (#1474) #1496

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

AMS21
Copy link
Contributor

@AMS21 AMS21 commented Nov 4, 2023

Every new commit or pull request will now be checked with clang-format.
Note that only violations in new code will be reported and any old code will stay just as it is.

Here is an example PR to showcase how this would look like.

@AMS21 AMS21 force-pushed the code_style/clang-format branch 2 times, most recently from 76ac8a8 to c604d26 Compare November 6, 2023 09:54
@AMS21 AMS21 force-pushed the code_style/clang-format branch from c604d26 to 305b51f Compare November 17, 2023 14:13
@AMS21 AMS21 force-pushed the code_style/clang-format branch from 305b51f to 7a4db46 Compare November 25, 2023 13:04
@AMS21 AMS21 force-pushed the code_style/clang-format branch from 7a4db46 to bc1bfa3 Compare December 4, 2023 11:00
@Xottab-DUTY Xottab-DUTY force-pushed the dev branch 2 times, most recently from 143d1ca to 118d39d Compare December 16, 2023 10:39
@AMS21 AMS21 force-pushed the code_style/clang-format branch from bc1bfa3 to 33d492d Compare March 15, 2024 10:16
@AMS21
Copy link
Contributor Author

AMS21 commented Mar 15, 2024

Rebased and update to LLVM 18

@Xottab-DUTY Xottab-DUTY force-pushed the dev branch 2 times, most recently from 5b2ec76 to 6fffce9 Compare May 4, 2024 03:27
@AMS21 AMS21 force-pushed the code_style/clang-format branch from 33d492d to a94e78c Compare May 14, 2024 16:19
@Xottab-DUTY Xottab-DUTY force-pushed the dev branch 2 times, most recently from e89fcc8 to f6fd5cc Compare October 24, 2024 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To do
Development

Successfully merging this pull request may close these issues.

2 participants