Skip to content

Format cpp,h,C files#48

Merged
gaede merged 3 commits into
iLCSoft:masterfrom
Victor-Schwan:fmt-macros
Jun 1, 2026
Merged

Format cpp,h,C files#48
gaede merged 3 commits into
iLCSoft:masterfrom
Victor-Schwan:fmt-macros

Conversation

@Victor-Schwan

Copy link
Copy Markdown
Contributor

BEGINRELEASENOTES

  • Add a .clang-format file using the LLVM style
  • Format all .cpp, .C and .h files
  • Add formatting commit hashes to .git-blame-ignore-revs

ENDRELEASENOTES

@jmcarcell

Copy link
Copy Markdown
Contributor

Why separate formatting macros and files? Can you putt all formatting in the same commit?
For the .clang-format did you pick the Key4hep one? We never decided anything for iLCSoft, but it would make it somewhat consistent (even if the format that has been used in the past in iLCSoft repositories is different). If one day we decide to have a common .clang-format in iLCSoft repositories then no extra formatting would be needed.

@Victor-Schwan

Copy link
Copy Markdown
Contributor Author

At first, I wanted to format the macros and only then I realized that I probably should format every relevant file in the repository. Ofc I can combine the commits.
I have used the LLVM style but I am fine with using which ever style you prefer. Assuming that the key4hep style is consistent in all repos, I would just copy the one from key4hep/k4FWCore, ok?

@gaede gaede merged commit 366ca63 into iLCSoft:master Jun 1, 2026
4 checks passed
@jmcarcell

Copy link
Copy Markdown
Contributor

Assuming that the key4hep style is consistent in all repos, I would just copy the one from key4hep/k4FWCore, ok?

Yes, that works, they are the same in most repos, or I think so.

@Victor-Schwan Victor-Schwan deleted the fmt-macros branch June 1, 2026 12:29
@Victor-Schwan Victor-Schwan mentioned this pull request Jun 4, 2026
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.

3 participants