Thanks Michal.
The config file is mostly just the default values with the only customisation coming in the first 5 lines:
- line 1 to run only the modernize-use-nullptr check
- line 3 to ensure that it warns about things in our header files
- line 5 so that it uses our .clang-format file for reformatting after any auto-applied changes
I have no doubt that the existing code will need cleaning up as we enable more checks. This is why I say "eventually" about having this run automatically, otherwise we'll just be swamped with warnings. A point for subsequent discussion will be what checks we want to use and in what order to start to gradually enable them and fix stuff as we go. I'll create a task where we can have such a discussion.