Remove unneeded note.

This commit is contained in:
Emma Nora Theuer 2025-01-01 20:48:25 +01:00
parent b954086f51
commit 1269e379a5

View file

@ -13,7 +13,7 @@ global logger
logger = logging.getLogger("wallman")
class _ConfigLib:
# Initializes the most important config values. TODO: Add handling for the empty config edge case
# Initializes the most important config values. TODO: Add handling for the empty config edge case and the FileNotFound case
def __init__(self) -> None:
self.config_file: ConfigFile = self._initialize_config() # Full config
# Dictionaries
@ -140,7 +140,7 @@ class ConfigValidity(_ConfigLib):
def _check_general_validity(self) -> bool:
# FIXME!
# HACK: Adjust it to check for the actually required variables existing rather than check if a number of options is set, which is highly error prone.
# TODO: Adjust it to check for the actually required variables existing rather than check if a number of options is set, which is highly error prone.
if len(self.config_general) < 3:
try:
self._set_fallback_wallpaper()
@ -211,7 +211,6 @@ class ConfigValidity(_ConfigLib):
class WallpaperLogic(_ConfigLib):
def __init__(self) -> None:
super().__init__()
# NOTE: This looks a bit ugly. Consider pros and cons of adding this into _ConfigLib
self.chosen_wallpaper_set: str = "fFTojkvCLIkGVlC6vUo4701djCZczmJDiyHKj4Qdj0zwkLyETsPxP88DLmY9In0I"
# NOTE: This function could be in a different file because it's not needed in the case only 1 wallpaper per set is needed.