ComPWA / ComPWA-legacy

[deprecated] C++ back-end for the Common Partial Wave Analysis framework

https://compwa.github.io/legacy

Other

9 stars 9 forks source link

style: implement and apply cmake-format #303

Closed redeboer closed 4 years ago

redeboer commented 4 years ago

See cmake-format, using v0.6.9.

wgradl commented 4 years ago

This seems to make a pig's ear of some of the comments, where the line breaks actually have been placed on purpose. Can this formatting tool be taught to leave comments alone?

redeboer commented 4 years ago

Good point, cmake-format reflows comments by default, I guess because it assumes they shouldn't affect CMake.

See option enable_markup, I set it to False now and reapplied, let's see if that improves things...

codecov[bot] commented 4 years ago

Codecov Report

Merging #303 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #303   +/-   ##
=======================================
  Coverage   36.59%   36.59%           
=======================================
  Files          88       88           
  Lines        7379     7379           
=======================================
  Hits         2700     2700           
  Misses       4679     4679

wgradl commented 4 years ago

Looks good to me, at least as far as the comments are concerned.