search

dsuni / svndumpsanitizer

A program aspiring to be a more advanced version of svndumpfilter
https://miria.homelinuxserver.org/svndumpsanitizer/
GNU General Public License v3.0
48 stars 15 forks source link

Normalize and automate formatting via `clang-format` #21

Closed lafrenierejm closed 9 months ago

lafrenierejm commented 9 months ago

This PR does three things:

  1. Defines a clang-format config that, in my testing, resulted in the minimum number of changes.
  2. Runs clang-format across the codebase as a one-time normalization.
  3. Adds a GitHub Actions workflow to run clang-format against each PR (via Nix), failing if the PR's code does not already adhere to the formatting rules.
lafrenierejm commented 9 months ago

@dsuni Any opinions on this PR? I know that formatting can be highly subjective, so it's totally fine if you don't want these changes.

dsuni commented 9 months ago

Ah, yes, sorry. Had other things and forgot to respond. I'm not a huge fan of autoformatters and the like, so I think I'll pass on this one, thanks.

lafrenierejm commented 9 months ago

All good! Thanks for giving it a look.