[Paraview-developers] [kwrobot] New `Do: reformat` command

Cory Quammen cory.quammen at kitware.com
Wed Mar 22 11:34:15 EDT 2017


Nice!

On Wed, Mar 22, 2017 at 11:26 AM, Ben Boeckel <ben.boeckel at kitware.com> wrote:
> Hi,
>
> The kwrobot can now automatically fix the `clang-format` errors detected
> for your MRs. Developers can run it on any MR and the submitter of an MR
> can run it on their own MRs as well. Note that this will push right back
> to your source branch, so you will need to fetch from your fork to sync
> up with the reformatted branch.
>
> This may be done by using the new command:
>
>     Do: reformat
>
> The topology of your MR is untouched, so merges and the like are
> preserved. If non-merge commits are found to be empty after a reformat
> (and not empty before reformatting), they are removed from the history
> since it is assumed they were "make the formatting checks pass" commits
> (dropped commits will be mentioned if it happens since it really is just
> a heuristic).
>
> --Ben
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Search the list archives at: http://markmail.org/search/?q=Paraview-developers
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/mailman/listinfo/paraview-developers



-- 
Cory Quammen
Staff R&D Engineer
Kitware, Inc.


More information about the Paraview-developers mailing list