Formatting checks disabled (possible erroneous?)

[aditya-vk]

Two issues:

1. Format checks are currently disabled. I think this was intentional during the shift to bionic and clang-format-6.0. We need to bring it back.

2. I am not sure if the following issue is local to me (@madan96 can you please verify). a) Master still seems to need formatting corrections. And, b) On running the formatting command, Combine InteractiveMarkerViewer classes. #521 seems to still fail. For instance src/rviz/InteractiveMarkerViewer.cpp Line 206 is never corrected in its indentation. I have clang-format-6.0 on bionic.

[madan96]

Yeah, I just checked and there seem to be some src files in rviz, perception and control subdirectories that still require formatting changes. Not sure how they got left out, there is no PR affecting these files, after #549 that was merged into master after this so I think that's where the problem is.

[madan96]

@brianhou Should we run check-format for all builds or at least BIONIC_CATKIN_FULL_RELEASE?

[brianhou]

We only need to check the formatting on one build.