================ @@ -18,98 +18,65 @@ """ import argparse +import github +import json import os import subprocess import sys -from typing import List, Optional +from typing import Any, Dict, Final, List, Sequence class LintArgs: - start_rev: str = None - end_rev: str = None - repo: str = None - changed_files: List[str] = [] - token: str = None + start_rev: str ---------------- EugeneZelenko wrote:
Is it necessary to expose this data member to external world? If not, prefix with two underscores should be uses (private). Same in other places. https://github.com/llvm/llvm-project/pull/168827 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
