Closed haruki7049 closed 1 month ago
Hi @haruki7049! Thanks for this PR. jank currently requires devs to format their C++ with clang-format
, but we don't have any other project-wide formatting. I'm not convinced we want this right now, especially tied to Nix. The vast majority of devs will not be using Nix.
I know you put a lot of work into this, which I very much appreciate. For now, I think we should close this; if we want project-wide formatting, in the future, this is where we should start.
To be sure that the work you're doing is going to be merged, assuming there's no ticket for it, you can always check with me. :bow:
PR for combining declarations of which formatter to use in the same file, so treefmt may be used instead of treefmt-nix.
How to use formatter
How to add formatter
Just add an attribute in flake.nix: