From 246892130fe4cea9ea67d8eb1ad1bbc20a579fbf Mon Sep 17 00:00:00 2001 From: Paul Keith Date: Sun, 4 Jun 2017 17:03:13 -0500 Subject: [PATCH] maintainers-checks: Use elif to check 'dd' install method * If a device doesn't have a valid install method, there's no point in checking if it has 'dd' as the install method, it won't match up anyways * This has no visible impact on the time of running this script, but the logic is now better Change-Id: I204903db40f02fe1ff837ed6feb8d06ace86cc92 --- maintainer-checks/maintainers.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/maintainer-checks/maintainers.py b/maintainer-checks/maintainers.py index 09b2312..458b61f 100755 --- a/maintainer-checks/maintainers.py +++ b/maintainer-checks/maintainers.py @@ -83,7 +83,7 @@ for codename in codenames: try: if not yml["install_method"] or "TODO" in yml["install_method"]: print("{} doesn't have an install method listed".format(codename)) - if "dd" in yml["install_method"]: + elif "dd" in yml["install_method"]: try: if not yml["recovery_partition"]: print("{} doesn't have a recovery partition listed".format(codename))