lava / matplotlib-cpp

Extremely simple yet powerful header-only C++ plotting library built on the popular matplotlib
MIT License
4.28k stars 1.12k forks source link

Introduce styleguide to avoid unnecessary merge conflicts #286

Open mightymichael44 opened 2 years ago

mightymichael44 commented 2 years ago

Hi @lava, thanks for the great project.

I just discovered that I would like to have both the master of this repo as well as the new features from #201
Since it has +/-1.5k lines it's clearly not mergeable, but while reading the changes I noticed a lot of the differences arise from different formatting...

What about introducing a stylefile, e.g. .clang-format file in whatever style you prefer and applying it here to the master.

Applying it on all incoming PR's would remove the clutter of different formatting.

P.S.: If you decide on a preferred codestyle I could open a PR, adding the .clang-format file and the formatted matplotlibcpp.h