daphne-eu / daphne

DAPHNE: An Open and Extensible System Infrastructure for Integrated Data Analysis Pipelines
Apache License 2.0
67 stars 62 forks source link

run clang-format on push / PR #833

Closed philipportner closed 1 month ago

philipportner commented 1 month ago

With the new .clang-format file we should make sure that new code also adheres to the coding style defined by clang-format. I don't think we should add something that automatically enforces formatting for every user, e.g., pre-commit hooks. Rather, I would suggest to add a CI action that simply fails when the code in the commit/PR does not follow our code style.

This can easily be done with https://github.com/marketplace/actions/clang-format-check

philipportner commented 1 month ago

closed with #835