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

chore: Unify .clang-format file #5037

Merged
merged 3 commits into from
Dec 17, 2023
Merged

Conversation

pajlada
Copy link
Member

@pajlada pajlada commented Dec 17, 2023

Description

  • Move all .clang-format files to the root directory
    Since they're all the same, there's no need to copy paste the same file multiple times.
    The only downside this has it that when opening a project under lib that does not have its own .clang-format file, your editor might try to format it based on our config.
  • Update scripts/check-format.sh to include all source dirs

@pajlada pajlada enabled auto-merge (squash) December 17, 2023 13:27
@pajlada
Copy link
Member Author

pajlada commented Dec 17, 2023

image

@pajlada pajlada merged commit 269e7a0 into master Dec 17, 2023
20 checks passed
@pajlada pajlada deleted the chore/unify-clang-format-file branch December 17, 2023 13:50
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