From 968f5aaf06421f5e9e861dea821dba977cd2fe85 Mon Sep 17 00:00:00 2001 From: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> Date: Tue, 31 Mar 2026 12:25:37 +0300 Subject: [PATCH] Replace for loop with list comprehension --- pyproject.toml | 1 + src/humanize/number.py | 9 ++------- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index 5cbff14..bcc94b4 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -65,6 +65,7 @@ lint.select = [ "ICN", # flake8-import-conventions "ISC", # flake8-implicit-str-concat "LOG", # flake8-logging + "PERF", # perflint "PGH", # pygrep-hooks "PIE", # flake8-pie "PT", # flake8-pytest-style diff --git a/src/humanize/number.py b/src/humanize/number.py index 77b79ab..4aeffcc 100644 --- a/src/humanize/number.py +++ b/src/humanize/number.py @@ -441,14 +441,9 @@ def scientific(value: NumberOrString, precision: int = 2) -> str: import re part2 = re.sub(r"^\+?(\-?)0*(.+)$", r"\1\2", part2) + part2_list = [exponents[char] for char in part2] - new_part2 = [] - for char in part2: - new_part2.append(exponents[char]) - - final_str = part1 + " x 10" + "".join(new_part2) - - return final_str + return part1 + " x 10" + "".join(part2_list) def clamp(