Oh I don't mind at all. Sounds reasonable.

However, in other projects, we leave it as a warning in debug builds, and have a verify configuration that tries to enforce all the rules that it can. In the verify build configuration, it would be an error to have no comments on a publicly visible member or type. Does that seem more reasonable? This way, as you're preparing to create a PR, you are reminded of the rules. Might scale a little better than explaining it in PRs.