From 5a17ab3ff3ad620110640eeef709f20701160ff2 Mon Sep 17 00:00:00 2001 From: Nikhil Komawar Date: Thu, 12 Oct 2017 16:47:29 -0400 Subject: tools: rename unused variale in more pythonic way The 'Main' function in tools/test.py file was using a variable named ``all_outcomes`` to store a value not being used. It is a best practice to name unused variables, often return values of functions/methods (as in this case) as ``_`` [1]. This just helps keep the code a bit cleaner and avoid any silly mistakes. PR-URL: https://github.com/nodejs/node/pull/16171 Refs: [1] https://stackoverflow.com/a/5477153 Reviewed-By: James M Snell Reviewed-By: Claudio Rodriguez Reviewed-By: Gireesh Punathil Reviewed-By: Colin Ihrig Reviewed-By: Ruben Bridgewater --- tools/test.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tools/test.py') diff --git a/tools/test.py b/tools/test.py index c2e00ace50..e5b3a550dd 100755 --- a/tools/test.py +++ b/tools/test.py @@ -1661,7 +1661,7 @@ def Main(): } test_list = root.ListTests([], path, context, arch, mode) unclassified_tests += test_list - (cases, unused_rules, all_outcomes) = ( + (cases, unused_rules, _) = ( config.ClassifyTests(test_list, env)) if globally_unused_rules is None: globally_unused_rules = set(unused_rules) -- cgit v1.2.3