We are happy that our CFSReader plugin got integrated and we very much appreciate the help and support of the kitware guys - with very special thanks to Mathieu but also to Ben and Cory.
I offer to add to the documentation some guidelines I learned during the review process. My idea is we briefly discuss here the approach and location, I would then make a suggestion via a merge request and leave it to you, if you are interested in using it or not.
I realized that the style guide of VTK’s develop.md is only in Ben’s branch but not on VTK’s master. I did not want to refer to a merge request and therefore created a file guidelines.md which could be extended/merged and referred also from VTK.
There is some more content in Ben’s branch but I did not want to copy it but partially the instructions overlap. For my description I skipped what is enforced via clang-format from the robot anyway.
Feel free to use the proposal, not use it or ask me for some changes
With respect to the guidelines, this is purely what I extracted from the reviews - it is not my ideas and actually quite different from my style of own code.
