[Paraview-developers] Code formatting checks

Ben Boeckel ben.boeckel at kitware.com
Thu Mar 2 15:05:47 EST 2017


Hi there,

Once this merge request has been merged:

    https://gitlab.kitware.com/paraview/paraview/merge_requests/1431

branches built on top of it will be checked for using `clang-format`.
This will only affect branches which are based on `master` after it has
been merged, so older branches may contain formatting problems that will
only show up when the file is changed in a future merge request.

For the next few weeks, once a week, I'll reformat `master` using the
scripts to help clean up any issues that creep in due to older branches,
but after that, branches should be rebased to just have this as part of
its history.

In the meantime, if you'd like to clean up formatting of your merge
requests, there is the `Utilities/Scripts/clang-format.bash` which
contains documentation on how to fix up a merge request. Note that
ParaView's configuration requires a newer `clang-format` (3.8).

We'll be testing the support of the robot to do reformatting for you if
you'd like, but it needs some more verification before we deploy it on
ParaView itself.

Thanks,

--Ben


More information about the Paraview-developers mailing list