aboutsummaryrefslogtreecommitdiffstats
path: root/packaging-tools/optionparser.py
diff options
context:
space:
mode:
authorPatrik Teivonen <patrik.teivonen@qt.io>2022-06-22 15:09:08 +0300
committerPatrik Teivonen <patrik.teivonen@qt.io>2022-08-16 09:23:54 +0000
commit25a59a540a652511f38425a31130dcd85b105166 (patch)
tree3d95c0c7238606a9e8f07d2ac4db598b90448743 /packaging-tools/optionparser.py
parent08d6d621822c803d41359348133bd37eae5f370d (diff)
pylint: Add string formatting checks, use f-string and lazy % formatting
Enable following checks in pylint: - C0209: Formatting a regular string which could be a f-string - W1202: Use lazy % formatting in logging functions - W1203: Use lazy % formatting in logging functions Use f-string for string formatting where recommended and lazy % formatting in logging. Change-Id: I14924d27e0f800c47af1ec903be4c51f24191d26 Reviewed-by: Antti Kokko <antti.kokko@qt.io>
Diffstat (limited to 'packaging-tools/optionparser.py')
-rw-r--r--packaging-tools/optionparser.py10
1 files changed, 5 insertions, 5 deletions
diff --git a/packaging-tools/optionparser.py b/packaging-tools/optionparser.py
index 853a826e7..2ceaff343 100644
--- a/packaging-tools/optionparser.py
+++ b/packaging-tools/optionparser.py
@@ -38,7 +38,7 @@ class PackagingOptions:
"""Utility class to read options from configuration file that follows .ini file format."""
def __init__(self, confFile):
if not os.path.isfile(confFile):
- raise IOError("Not a valid file: {0}".format(confFile))
+ raise IOError(f"Not a valid file: {confFile}")
self.config = ConfigParser(os.environ)
self.config.optionxform = str
self.config.read(confFile)
@@ -50,9 +50,9 @@ class PackagingOptions:
try:
dict1[option] = self.config.get(section, option)
if dict1[option] == -1:
- print("skip: %s" % option)
+ print(f"skip: {option}")
except Exception:
- print("exception on %s!" % option)
+ print(f"exception on {option}!")
dict1[option] = None
return dict1
@@ -70,10 +70,10 @@ class PackagingOptions:
def verbose(self):
for section in self.config.sections():
- print("[" + section + "]")
+ print(f"[{section}]")
options = self.config.options(section)
for option in options:
- print("{0} = {1}".format(option, self.config.get(section, option)))
+ print(f"{option} = {self.config.get(section, option)}")
print