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.

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

Local test PR for Pre-commit CI #64

Open
wants to merge 6 commits into
base: testPrTarget
Choose a base branch
from
Open

Conversation

ursg
Copy link
Owner

@ursg ursg commented Sep 21, 2023

If all goes well, this should fix the final line of vlasiator.cpp with an automagic commit.

This commit contains a config file for pre-commit, which automatically
adjusts whitespace and formatting for source files. It *also* contains
all the changes this config would enact onto the source tree, fixing all
trailing whitespace problems and tab/space conversion in the source.
This is a big mess, so I suggest delaying this PR and re-creating it at
some big merge point.

Note that the pre-commit config also contains a commented-out hook for
clang-format according to the clang-format config file already in the
repository, but I consider it too aggressive to automatically enforce it
everywhere.
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.

1 participant