-
Notifications
You must be signed in to change notification settings - Fork 23
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
Respect Skipping CMake Formatting Based on Setting in Top Level Project #35
Comments
Just realized that one option is to just use |
Currently I don't think per-project formatting rules are supported. The recommended usage is to add |
Ok no problem, we can work around this for now. And yes, I've seen that strategy already in your ModernCppStarter, but the issue would still remain if we want to disable CMake formatting in a downstream project if it's enabled upstream. |
I'm not sure how this would work, or if it's technically feasible, but I'm going to describe my issue through a hypothetical project.
Let's assume we have projects A and B. Project A depends on project B and pulls it in with
CPM
.Both projects use
Format.cmake
; project A does not allow Cmake formatting, but project B does have that enabled.Right now, when running "check-format" from project A, it will always fail because
Format.cmake
thinks that the CMakeLists files need to be formated even thoughFORMAT_SKIP_CMAKE
is set toYES
in project A.Is there any way to work around this?
The text was updated successfully, but these errors were encountered: