Fix various static analysis issues

Author: impredicative

ircurltitlebot/__init__.py

@@ -1 +1,2 @@
+"""Package initialization."""
 from .bot import Bot

ircurltitlebot/__main__.py

@@ -1,6 +1,7 @@
+"""Package entrypoint."""
 import argparse
-import logging
 import json
+import logging
 from pathlib import Path
 
 from ruamel.yaml import YAML
@@ -11,48 +12,52 @@
 
 
 def main() -> None:
+    """Start the bot."""
     # Read args
     parser = argparse.ArgumentParser(prog=config.PACKAGE_NAME, description="IRC URL title posting bot")
-    parser.add_argument('--config-path', required=True, help='Configuration file path, e.g. /some/dir/config.yaml')
+    parser.add_argument("--config-path", required=True, help="Configuration file path, e.g. /some/dir/config.yaml")
     instance_config_path = Path(parser.parse_args().config_path)
 
     # Read user config
-    log.debug('Reading instance configuration file %s', instance_config_path)
+    log.debug("Reading instance configuration file %s", instance_config_path)
     instance_config = YAML().load(instance_config_path)
     instance_config = json.loads(json.dumps(instance_config))  # Recursively use a dict as the data structure.
 
     # Log user config
     logged_instance_config = instance_config.copy()
-    if 'sites' in logged_instance_config:
-        del logged_instance_config['sites']
-    log.info('Read user configuration file %s having excerpted configuration: %s',
-             instance_config_path, logged_instance_config)
-    for site, site_config in instance_config.get('sites', {}).items():
-        log.info('User configuration for site %s is: %s', site, site_config)
+    if "sites" in logged_instance_config:
+        del logged_instance_config["sites"]
+    log.info(
+        "Read user configuration file %s having excerpted configuration: %s",
+        instance_config_path,
+        logged_instance_config,
+    )
+    for site, site_config in instance_config.get("sites", {}).items():
+        log.info("User configuration for site %s is: %s", site, site_config)
 
     # Set alerts channel
-    if 'alerts_channel' not in instance_config:
-        instance_config['alerts_channel'] = config.ALERTS_CHANNEL_FORMAT_DEFAULT
-    instance_config['alerts_channel'] = instance_config['alerts_channel'].format(nick=instance_config['nick'])
-    if instance_config['alerts_channel'] not in instance_config['channels']:
-        instance_config['channels'].append(instance_config['alerts_channel'])
+    if "alerts_channel" not in instance_config:
+        instance_config["alerts_channel"] = config.ALERTS_CHANNEL_FORMAT_DEFAULT
+    instance_config["alerts_channel"] = instance_config["alerts_channel"].format(nick=instance_config["nick"])
+    if instance_config["alerts_channel"] not in instance_config["channels"]:
+        instance_config["channels"].append(instance_config["alerts_channel"])
 
     # Process user config
-    instance_config['nick:casefold'] = instance_config['nick'].casefold()
-    instance_config['channels:casefold'] = [channel.casefold() for channel in instance_config['channels']]
-    instance_config['ignores:casefold'] = [ignore.casefold() for ignore in instance_config.get('ignores', [])]
+    instance_config["nick:casefold"] = instance_config["nick"].casefold()
+    instance_config["channels:casefold"] = [channel.casefold() for channel in instance_config["channels"]]
+    instance_config["ignores:casefold"] = [ignore.casefold() for ignore in instance_config.get("ignores", [])]
 
     # Process blacklist
-    blacklists = instance_config['blacklist'] = instance_config.get('blacklist', {})
-    blacklists['title'] = set(blacklists.get('title', set()))
-    blacklists['title'] = {entry.casefold() for entry in blacklists['title']}
-    blacklists['url'] = set(blacklists.get('url', set()))
+    blacklists = instance_config["blacklist"] = instance_config.get("blacklist", {})
+    blacklists["title"] = set(blacklists.get("title", set()))
+    blacklists["title"] = {entry.casefold() for entry in blacklists["title"]}
+    blacklists["url"] = set(blacklists.get("url", set()))
 
     config.INSTANCE = instance_config
 
     # Start bot
     Bot()
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     main()