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.

Sign up for GitHub

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

Jump to bottom

Add clang-format to autoformate C/C++ code #125

Merged
merged 14 commits into from
May 28, 2024
Merged

Add clang-format to autoformate C/C++ code #125

merged 14 commits into from
May 28, 2024

Conversation

boulderdaze
Copy link
Collaborator

@boulderdaze boulderdaze commented May 21, 2024 •
edited

Closes #111

  • Added .clang-format
  • Set up github actions using clang_format.yml

There is one change I made for musica clang-format that is different from micm.

  • micm :IndentPPDirectives: AfterHash
#if FOO
#  if BAR
#    include <foo>
#  endif
#endif
  • musica: IndentPPDirectives: BeforeHash
#if FOO
  #if BAR
    #include <foo>
  #endif
#endif

@boulderdaze boulderdaze changed the title Add clang format to autoformat C++ heade Add clang format to autoformat C++ header May 21, 2024
@boulderdaze boulderdaze changed the title Add clang format to autoformat C++ header Add clang-format to autoformat C++ header May 21, 2024
@boulderdaze boulderdaze marked this pull request as ready for review May 22, 2024 19:43
@boulderdaze boulderdaze changed the title Add clang-format to autoformat C++ header Add clang-format to autoformate C/C++ code May 22, 2024
@boulderdaze boulderdaze merged commit 9851a1e into main May 28, 2024
51 of 55 checks passed
@boulderdaze boulderdaze deleted the 111-clang-format branch May 28, 2024 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@K20shores K20shores K20shores approved these changes

@mattldawson mattldawson Awaiting requested review from mattldawson

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add clang format to autoformat C++ header
2 participants
@boulderdaze @K20shores