precice / calculix-adapter

preCICE-adapter for the CSM code CalculiX
GNU General Public License v3.0
52 stars 20 forks source link

Add support for clang-format #74

Closed MakisH closed 2 years ago

MakisH commented 2 years ago

We need some automatic formatting to maintain the history clean and understandable.

This adds the following:

Closes #69.

Note that merging this could cause some merge conflicts in already open merge requests.

KyleDavisSA commented 2 years ago

Do you think it would be better to merge this first, and fix any other merge issues it might cause, or wait until the others are merged?

MakisH commented 2 years ago

Do you think it would be better to merge this first, and fix any other merge issues it might cause, or wait until the others are merged?

This is really easy to reapply, so I would wait for #73 first.

MakisH commented 2 years ago

I would actually wait until we merge the v2.17 to develop first.

MakisH commented 2 years ago

Closing in favor of #83.