Mark all class variables as Final

diff --git a/mkosi/installer/apt.py b/mkosi/installer/apt.py
index c576a67f456e64bc58a84951c3ffa236d1f69afb..a111dd25bc538494911e4441e8c73192a0013826 100644 (file)
--- a/mkosi/installer/apt.py
+++ b/mkosi/installer/apt.py
@@ -4,7 +4,7 @@ import dataclasses
 import textwrap
 from collections.abc import Sequence
 from pathlib import Path
-from typing import Optional
+from typing import Final, Optional
 
 from mkosi.config import PACKAGE_GLOBS, Config, ConfigFeature
 from mkosi.context import Context
@@ -37,7 +37,7 @@ class AptRepository:
 
 
 class Apt(PackageManager):
-    documentation_exclude_globs = [
+    documentation_exclude_globs: Final[list[str]] = [
         "usr/share/doc/*",
         "usr/share/man/*",
         "usr/share/groff/*",

diff --git a/mkosi/log.py b/mkosi/log.py
index e264c4decb1e2056d82f69099ebb6197dbe4415a..15fddad47eac85f97167b91b2be40ade3ef5e2ba 100644 (file)
--- a/mkosi/log.py
+++ b/mkosi/log.py
@@ -6,7 +6,7 @@ import logging
 import os
 import sys
 from collections.abc import Iterator
-from typing import Any, NoReturn, Optional
+from typing import Any, Final, NoReturn, Optional
 
 # This global should be initialized after parsing arguments
 ARG_DEBUG = contextvars.ContextVar("debug", default=False)
@@ -15,12 +15,12 @@ LEVEL = 0
 
 
 class Style:
-    bold = "\033[0;1;39m" if sys.stderr.isatty() else ""
-    blue = "\033[0;1;34m" if sys.stderr.isatty() else ""
-    gray = "\033[0;38;5;245m" if sys.stderr.isatty() else ""
-    red = "\033[31;1m" if sys.stderr.isatty() else ""
-    yellow = "\033[33;1m" if sys.stderr.isatty() else ""
-    reset = "\033[0m" if sys.stderr.isatty() else ""
+    bold: Final[str] = "\033[0;1;39m" if sys.stderr.isatty() else ""
+    blue: Final[str] = "\033[0;1;34m" if sys.stderr.isatty() else ""
+    gray: Final[str] = "\033[0;38;5;245m" if sys.stderr.isatty() else ""
+    red: Final[str] = "\033[31;1m" if sys.stderr.isatty() else ""
+    yellow: Final[str] = "\033[33;1m" if sys.stderr.isatty() else ""
+    reset: Final[str] = "\033[0m" if sys.stderr.isatty() else ""
 
 
 def die(message: str,

diff --git a/mkosi/partition.py b/mkosi/partition.py
index 41d243f504c156138dfc55d539ff83f9ef0841d2..58949216ecadbcca8237549353aa9d9f9d3af50d 100644 (file)
--- a/mkosi/partition.py
+++ b/mkosi/partition.py
@@ -3,7 +3,7 @@ import json
 import subprocess
 from collections.abc import Mapping, Sequence
 from pathlib import Path
-from typing import Any, Optional
+from typing import Any, Final, Optional
 
 from mkosi.log import die
 from mkosi.run import SandboxProtocol, nosandbox, run
@@ -27,7 +27,7 @@ class Partition:
             roothash=dict.get("roothash"),
         )
 
-    GRUB_BOOT_PARTITION_UUID = "21686148-6449-6e6f-744e-656564454649"
+    GRUB_BOOT_PARTITION_UUID: Final[str] = "21686148-6449-6e6f-744e-656564454649"
 
 
 def find_partitions(image: Path, *, sandbox: SandboxProtocol = nosandbox) -> list[Partition]:

diff --git a/mkosi/versioncomp.py b/mkosi/versioncomp.py
index 56da1ad2fb90b9c97835779d1057f2d58e31f20f..41619c33d9d4a4c3af50dcfa334d1103b6ef8f85 100644 (file)
--- a/mkosi/versioncomp.py
+++ b/mkosi/versioncomp.py
@@ -3,6 +3,7 @@
 import functools
 import itertools
 import string
+from typing import Final
 
 
 @functools.total_ordering
@@ -10,9 +11,9 @@ class GenericVersion:
     # These constants follow the convention of the return value of rpmdev-vercmp that are followe
     # by systemd-analyze compare-versions when called with only two arguments (without a comparison
     # operator), recreated in the compare_versions method.
-    _EQUAL = 0
-    _RIGHT_SMALLER = 1
-    _LEFT_SMALLER = -1
+    _EQUAL: Final[int] = 0
+    _RIGHT_SMALLER: Final[int] = 1
+    _LEFT_SMALLER: Final[int] = -1
 
     def __init__(self, version: str):
         self._version = version