[PATCH] D132550: Changes to code ownership in clang and clang-tidy

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman updated this revision to Diff 457289.
aaron.ballman marked an inline comment as done.
aaron.ballman added a comment.
Herald added subscribers: sstefan1, arphaman.
Herald added a reviewer: jdoerfert.

Because I'm touching this file so much anyway to add the GitHub handles, I had some ideas I wanted to float to see if others thought they were worthwhile:

1. I'd like to rearrange the file so that we don't organize it by surname (slightly problematic given that not everyone has a surname or goes by their given name anyway) but instead organize it by component. I think most of the people who are going to be interested in the contents of this file are going to be looking for a component to find a name instead of looking for a name to find a component.
2. I'd like to convert the file to RST format. This not only gives us some nicer ways to categorize the file contents, but it also makes it easier to publish this file on the website along with our other developer policy documentation.
3. It gives us a nice way to organize former code owners without making it more difficult to find active code owners (I did not try to find all former code owners)

This latest patch shows you what I have in mind for the changes. I'm not strongly tied to making the file into RST as that does reduce the readability of the text file somewhat, but I think the benefits to making this documentation more public outweigh the downsides to using markup.

(If someone knows of a way to get RST to style headings that are cut off the local table of contents due to the `:depth:` annotation, that would be lovely. Right now, the third level headings are as visually distinct as I'd like because the local table of contents is limited to two levels of nesting.)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132550/new/

https://reviews.llvm.org/D132550

Files:
  clang-tools-extra/CODE_OWNERS.TXT
  clang/CODE_OWNERS.TXT
  clang/CodeOwners.rst
  clang/docs/CodeOwners.rst
  clang/docs/index.rst

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132550.457289.patch
Type: text/x-patch
Size: 10250 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220901/3720ff4a/attachment-0001.bin>