llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: Charles Zablit (charles-zablit) <details> <summary>Changes</summary> `skipped_categories_list` was unused in `dotest.py`. This patch ensures it's used where intended. 2238dcc39358353cac21df75c3c3286ab20b8f53 switched to using `configuration.skip_categories` directly when reformatting the code, making `skipped_categories_list` unused. --- Full diff: https://github.com/llvm/llvm-project/pull/169903.diff 1 Files Affected: - (modified) lldb/packages/Python/lldbsuite/test/dotest.py (+1-5) ``````````diff diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py index 63f7df4de1894..f280bd2b3887b 100644 --- a/lldb/packages/Python/lldbsuite/test/dotest.py +++ b/lldb/packages/Python/lldbsuite/test/dotest.py @@ -1108,11 +1108,7 @@ def run_suite(): checkDAPSupport() skipped_categories_list = ", ".join(configuration.skip_categories) - print( - "Skipping the following test categories: {}".format( - configuration.skip_categories - ) - ) + print(f"Skipping the following test categories: {skipped_categories_list}") for testdir in configuration.testdirs: for dirpath, dirnames, filenames in os.walk(testdir): `````````` </details> https://github.com/llvm/llvm-project/pull/169903 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
