fix: Fix some linter errors (#193)

in_game_messages/discord_messaging.py

@@ -42,7 +42,7 @@ async def send_discord_message(
         self,
         channel: discord.TextChannel,
         msg: Dict,
-        parent: Dict = None,
+        parent: Optional[Dict] = None,
         parent_msg: Optional[discord.Message] = None,
         parent_thread: Optional[discord.Thread] = None,
     ) -> Tuple[discord.Message, Optional[discord.Thread]]:
@@ -63,9 +63,7 @@ async def send_discord_message(
                         parent_msg.id,
                     )
 
-                    thread_name = (
-                        f'Turn {parent["turn"]} - {parent["sender"]["name"]}'  # type: ignore # noqa
-                    )
+                    thread_name = f'Turn {parent["turn"]} - {parent["sender"]["name"]}'  # type: ignore # noqa
                     parent_message = await channel.fetch_message(parent_msg.id)
                     parent_thread = await parent_message.create_thread(name=thread_name)
                     if self.discord_user_ids:
@@ -81,7 +79,7 @@ async def send_discord_message(
             raise
 
     async def send_new_messages_to_discord(
-        self, messages: List, planets_game_id: str, mbox_path: Path = None
+        self, messages: List, planets_game_id: str, mbox_path: Optional[Path] = None
     ) -> None:
         """Fetch messages from a game and send new ones to Discord."""
         if not mbox_path:

in_game_messages/mbox.py

@@ -6,7 +6,7 @@
 import logging
 import mailbox
 from pathlib import Path
-from typing import Dict
+from typing import Dict, Optional
 
 
 class Mbox:
@@ -31,7 +31,7 @@ def get_message_ids(self) -> Dict:
     def save_email_message(
         self,
         message: Dict,
-        thread_id: str = None,
+        thread_id: Optional[str] = None,
     ) -> bool:
         """Construct an e-mail from an in-game message and save it to a mailbox."""
         try:

in_game_messages/messaging.py

@@ -7,7 +7,7 @@
 import re
 import struct
 from datetime import datetime, timedelta, timezone
-from typing import Dict, List
+from typing import Dict, List, Optional
 
 import requests
 
@@ -132,7 +132,7 @@ def get_message_participants(self, message: Dict) -> Dict:
 
     @staticmethod
     def construct_outlook_thread_index(
-        message: Dict, parent_thread_index: str = None
+        message: Dict, parent_thread_index: Optional[str] = None
     ) -> str:
         """Construct a thread index to allow threading in Outlook."""
         thread_index = ""

in_game_messages/slack_messaging.py

@@ -5,7 +5,7 @@
 import re
 import time
 from pathlib import Path
-from typing import Dict, List
+from typing import Dict, List, Optional
 
 from slack_sdk import WebClient
 from slack_sdk.errors import SlackApiError
@@ -29,7 +29,7 @@ def __init__(
         self.logger = logging.getLogger(__name__)
 
     def send_slack_message(
-        self, text: str, sender: Dict, parent: str = None
+        self, text: str, sender: Dict, parent: Optional[str] = None
     ) -> SlackResponse:
         """Send Slack message to the configured channel."""
         try:
@@ -70,7 +70,7 @@ def send_slack_message(
             raise
 
     def send_new_messages_to_slack(
-        self, messages: List, planets_game_id: str, mbox_path: Path = None
+        self, messages: List, planets_game_id: str, mbox_path: Optional[Path] = None
     ) -> None:
         """Fetch messages from a game and send new ones to Slack."""
         if not mbox_path:

in_game_messages/version.py

@@ -1 +1,3 @@
+"""In-game-messages version number."""
+
 __version__ = "0.3.3"