uber / h3

Hexagonal hierarchical geospatial indexing system
https://h3geo.org
Apache License 2.0
4.95k stars 470 forks source link

Run cmake-format on all CMake files #936

Open isaacbrodsky opened 1 month ago

isaacbrodsky commented 1 month ago

We could choose to enforce this in CI, but not sure the format is sufficiently better to do that & require everyone run it

Here is the command used:

cmake-format CMakeLists.txt CMakeTests.cmake cmake/Config.cmake.in cmake/TestWrapValgrind.cmake tests/cli/*.txt -i
dfellis commented 1 month ago

This PR misses all of the cmake text in /tests/cli. Perhaps we should rename them to *.cmake and modify the macro embedding them to use that new extension instead of *.txt?

coveralls commented 1 month ago

Coverage Status

coverage: 98.782%. remained the same when pulling a62a597aeb91f390c0621a57f23b68727d0faac6 on isaacbrodsky:cmake-format into 5d5ad33901e42550aa3dd310c96592b5298a5dbb on uber:master.