Skip to content

Add auto formatting and pre-commit #404

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jbreue16
Copy link
Contributor

@jbreue16 jbreue16 commented May 6, 2025

As per discussion in #402

Todo

@schmoelder
Copy link
Contributor

schmoelder commented May 6, 2025

Note, for now the PR only includes a .pre-commit file. This only performs a very limited number of formatting steps (e.g. removing trailing whitespace). clang-format still would need to be configured (but could also be added to the pre-commit hook.

Note, we also need to add instructions on how to install pre-commit, e.g.:

sudo apt install pre-commit

and how to activate it:

pre-commit install

@schmoelder
Copy link
Contributor

PR is probably also a duplicate of #148

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

Use clang-format for formatting code
2 participants