Cut the Gordian knot of coding style
Marco Barisione has written a tool to automatically enforce coding style only on the lines that you commit: https://blog.barisione.org/2018/06/06/using-clang-format-only-on-newly-written-code/
This is probably a good excuse to clean up our C++ coding style. Even when GJS was still written in C it never followed the GNOME style, and since switching to more C++ features it has only become more idiosyncratic and organically grown. We have cpplint which enforces Google C++ style, but which we disable some of the rules. Clang-format will also not perfectly replicate our current C++ style. I think we should stop fighting the tools and adopt Google C++ style, and just use a configuration that's close to default, and use the clang-format hooks to make sure that all new code conforms from now on.
This will hopefully take the frustration out of trying to "beat" the cpplint job, and remove all considerations of mechanical coding style from code reviews; if it passes the commit hook and the cpplint job, then it's correct.