Skip to content

Conversation

fbaeuerle
Copy link
Contributor

@fbaeuerle fbaeuerle commented Apr 24, 2025

Fix the indentation of the C++ copyright template in order to comply with the Eclipse handbook

@nradakovic
Copy link
Member

If we now decide to do it, we will need to update all copyright headers inside all C/C++ files inside project. Is this mandatory change?

@fbaeuerle
Copy link
Contributor Author

fbaeuerle commented May 20, 2025

Well, it is not strictly mandatory, but looking at e.g. https://github.com/eclipse-score/baselibs, most of the c++ code uses the formatting that this PR introduces. I suppose in the end we need a formatting that is compatible with clang-format and enforce both the copyright template check and clang-format.

Fix the indentation of the C++ copyright template.
@fbaeuerle fbaeuerle force-pushed the fb/fix_cpp_cr_template branch from ecf8562 to 8d37a06 Compare October 10, 2025 10:50
@fbaeuerle fbaeuerle marked this pull request as ready for review October 10, 2025 10:52
@nradakovic nradakovic merged commit ac35346 into eclipse-score:main Oct 15, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from Draft to Done in Infrastructure Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants