diff options
author | Patrik Teivonen <patrik.teivonen@qt.io> | 2022-06-22 15:09:08 +0300 |
---|---|---|
committer | Patrik Teivonen <patrik.teivonen@qt.io> | 2022-08-16 09:23:54 +0000 |
commit | 25a59a540a652511f38425a31130dcd85b105166 (patch) | |
tree | 3d95c0c7238606a9e8f07d2ac4db598b90448743 /packaging-tools/optionparser.py | |
parent | 08d6d621822c803d41359348133bd37eae5f370d (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.py | 10 |
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 |