Improve type definitions

Signed-off-by: Hiroshi Miura <miurahr@linux.com>

Author: miurahr

aqt/archives.py

@@ -451,11 +451,11 @@ def _get_archives_base(self, name, target_packages):
 
         self._parse_update_xmls(update_xmls, target_packages)
 
-    def _download_update_xml(self, update_xml_path: str, silent=False) -> Optional[str]:
+    def _download_update_xml(self, update_xml_path: str, silent: bool=False) -> Optional[str]:
         """Hook for unit test."""
         if not Settings.ignore_hash:
             try:
-                xml_hash = get_hash(update_xml_path, Settings.hash_algorithm, self.timeout)
+                xml_hash: Optional[bytes] = get_hash(update_xml_path, Settings.hash_algorithm, self.timeout)
             except ChecksumDownloadFailure:
                 if silent:
                     return None
@@ -470,7 +470,7 @@ def _download_update_xml(self, update_xml_path: str, silent=False) -> Optional[s
             xml_hash = None
         return getUrl(posixpath.join(self.base, update_xml_path), self.timeout, xml_hash)
 
-    def _parse_update_xml(self, os_target_folder, update_xml_text, target_packages: Optional[ModuleToPackage]):
+    def _parse_update_xml(self, os_target_folder: str, update_xml_text: str, target_packages: Optional[ModuleToPackage]) -> None:
         if not target_packages:
             target_packages = ModuleToPackage({})
         update_xml = Updates.fromstring(self.base, update_xml_text)
@@ -510,7 +510,7 @@ def _parse_update_xml(self, os_target_folder, update_xml_text, target_packages:
                     )
                 )
 
-    def _parse_update_xmls(self, update_xmls, target_packages: Optional[ModuleToPackage]):
+    def _parse_update_xmls(self, update_xmls: list[UpdateXmls], target_packages: Optional[ModuleToPackage]) -> None:
         if not target_packages:
             target_packages = ModuleToPackage({})
         for update_xml in update_xmls:
@@ -701,7 +701,7 @@ def handle_missing_updates_xml(self, e: ArchiveDownloadError):
     def _get_archives(self):
         self._get_archives_base(self.tool_name, None)
 
-    def _parse_update_xml(self, os_target_folder, update_xml_text, *ignored):
+    def _parse_update_xml(self, os_target_folder: str, update_xml_text: str, *ignored) -> None:
         update_xml = Updates.fromstring(self.base, update_xml_text)
         self._append_tool_update(os_target_folder, update_xml, self.arch, self.tool_version_str)