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

Add option to skip clang-format execution #36

Merged

Conversation

krypty
Copy link
Contributor

@krypty krypty commented Sep 29, 2023

Hello,

This option allows to use Format.cmake to only format CMake files (e.g. in the case C++ files formatting is handled by other means).

Please tell if I got any wrong with regards to contributing to this project.

Copy link
Owner

@TheLartians TheLartians left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR and implementing this feature! One small thing: could you run cmake-format on this code itself?

CMakeLists.txt Outdated Show resolved Hide resolved
This option allows to use Format.cmake to only format CMake files (e.g.
in the case C++ files formatting is handled by other means).
@krypty krypty force-pushed the add-option-skip-format-clang branch from 06c5a66 to 545711c Compare October 9, 2023 09:35
Copy link
Owner

@TheLartians TheLartians left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes!

@TheLartians TheLartians merged commit 734e26f into TheLartians:master Oct 9, 2023
3 checks passed
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.

2 participants