Add a CMake linter to prevent style change commits

McMartin / FRUT

Building JUCE projects using CMake made easy

https://frut.readthedocs.io

GNU General Public License v3.0

421 stars 40 forks source link

Add a CMake linter to prevent style change commits #29

Open MartyLake opened 7 years ago

MartyLake commented 7 years ago

A quick google search got me this result, maybe others are more appropriate ?

https://pypi.python.org/pypi/cmakelint/1.3.4

McMartin commented 7 years ago

There is also https://pypi.python.org/pypi/polysquare-cmake-linter/0.0.5

Xav83 commented 4 years ago

Hi @MartyLake @McMartin,

I don't know if this issue is still relevant, but the tool cmake-format may interest you for this. We use actually it in Expressive projects, and it works really nicely 😉

McMartin commented 4 years ago

Hi @Xav83,

Thanks for the pointer!

I ran cmake-format on the CMakeLists.txt files inside FRUT, and it found a few things that could be improved, so I opened https://github.com/McMartin/FRUT/pull/603 with these improvements.

I also tried running cmake-format on Reprojucer.cmake, but it seemed to hang... I guess formatting more than 6000 lines is hard 😅