[clang-tools-extra] [llvm] [Github][CI] Add `doc8` for clang-tidy documentation formatting (PR #168827)

[via cfe-commits]

================
@@ -34,82 +35,53 @@ class LintArgs:
     issue_number: int = 0
     build_path: str = "build"
     clang_tidy_binary: str = "clang-tidy"
+    doc8_binary: str = "doc8"
 
-    def __init__(self, args: argparse.Namespace = None) -> None:
+    def __init__(self, args: argparse.Namespace) -> None:
         if not args is None:
             self.start_rev = args.start_rev
             self.end_rev = args.end_rev
             self.repo = args.repo
             self.token = args.token
-            self.changed_files = args.changed_files
+            if args.changed_files:
+                self.changed_files = args.changed_files.split(",")
+            else:
+                self.changed_files = []
----------------
zeyi2 wrote:

> Is this needed? If empty `changed_files` are passed, then `self.changed_files` would be empty anyway

It was added because previously `doc8` and `clang-tidy` used different ways to handle `changed_files`, this was meant to unify them. But you are correct, this is not necessary. Thanks!

https://github.com/llvm/llvm-project/pull/168827