Fixed for type: ignore[arg-type]

Signed-off-by: Dimitris Zlatanidis <d.zlatanidis@gmail.com>
This commit is contained in:
Dimitris Zlatanidis 2024-04-22 22:39:07 +03:00
parent e10a842034
commit 6ef920918a
7 changed files with 31 additions and 25 deletions

View file

@ -59,15 +59,15 @@ class CheckUpdates(Configs):
def save_the_compares(self, repo: str) -> None: def save_the_compares(self, repo: str) -> None:
local_chg_txt: Path = Path( local_chg_txt: Path = Path(
self.repos.repositories[repo]['path'], # type: ignore[arg-type] self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['changelog_txt'] # type: ignore[arg-type] self.repos.repositories[repo]['changelog_txt']
) )
repo_chg_txt: str = ( repo_chg_txt: str = (
f"{self.repos.repositories[repo]['mirror_changelog']}" f"{self.repos.repositories[repo]['mirror_changelog']}"
f"{self.repos.repositories[repo]['changelog_txt']}" f"{self.repos.repositories[repo]['changelog_txt']}"
) )
repo_data_file: Path = Path(self.repos.repositories[repo]['path'], # type: ignore[arg-type] repo_data_file: Path = Path(self.repos.repositories[repo]['path'],
self.repos.data_json) self.repos.data_json)
if not repo_data_file.is_file(): if not repo_data_file.is_file():

View file

@ -95,7 +95,7 @@ class DownloadOnly(Configs):
self.asc_files.append(asc_file) self.asc_files.append(asc_file)
def copy_slackbuild_scripts(self, name: str) -> None: def copy_slackbuild_scripts(self, name: str) -> None:
repo_path_package: Path = Path(self.repos.repositories[self.repository]['path'], self.data[name]['location'], name) # type: ignore[arg-type] repo_path_package: Path = Path(self.repos.repositories[self.repository]['path'], self.data[name]['location'], name)
if not Path(self.directory, name).is_dir(): if not Path(self.directory, name).is_dir():
shutil.copytree(repo_path_package, Path(self.directory, name)) shutil.copytree(repo_path_package, Path(self.directory, name))

View file

@ -116,10 +116,10 @@ class MultiProcess(Configs):
self._error_process() self._error_process()
raise SystemExit(process.returncode) raise SystemExit(process.returncode)
def _error_process(self): def _error_process(self) -> None:
""" Prints error message for a process. """ """ Prints error message for a process. """
if not self.progress_bar_conf and not self.option_for_progress_bar: if not self.progress_bar_conf and not self.option_for_progress_bar:
message: str = 'Error occurred with process. Please check the log file.' # type: ignore[annotation-unchecked] message: str = 'Error occurred with process. Please check the log file.'
print() print()
print(len(message) * '=') print(len(message) * '=')
print(f'{self.bred}{message}{self.endc}') print(f'{self.bred}{message}{self.endc}')

View file

@ -16,6 +16,10 @@ from slpkg.utilities import Utilities
from slpkg.toml_errors import TomlErrors from slpkg.toml_errors import TomlErrors
def default_repositories():
return {}
@dataclass @dataclass
class Repositories: class Repositories:
toml_errors = TomlErrors() toml_errors = TomlErrors()
@ -240,7 +244,7 @@ class Repositories:
toml_errors.raise_toml_error_message(error, repositories_toml_file) toml_errors.raise_toml_error_message(error, repositories_toml_file)
# Dictionary configurations of repositories. # Dictionary configurations of repositories.
repositories = { repositories: dict[str, str] = {
sbo_repo_name: { sbo_repo_name: {
'enable': sbo_repo, 'enable': sbo_repo,
'path': sbo_repo_path, 'path': sbo_repo_path,

View file

@ -62,13 +62,13 @@ class UpdateRepositories(Configs):
""" Updates the binary repositories. """ """ Updates the binary repositories. """
urls: dict = {} urls: dict = {}
self.utils.create_directory(self.repos.repositories[repo]['path']) # type: ignore[arg-type] self.utils.create_directory(self.repos.repositories[repo]['path'])
self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'], # type: ignore[arg-type] self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['changelog_txt']) # type: ignore[arg-type] self.repos.repositories[repo]['changelog_txt'])
self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'], # type: ignore[arg-type] self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['packages_txt']) # type: ignore[arg-type] self.repos.repositories[repo]['packages_txt'])
self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'], # type: ignore[arg-type] self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['checksums_md5']) # type: ignore[arg-type] self.repos.repositories[repo]['checksums_md5'])
changelog: str = (f"{self.repos.repositories[repo]['mirror_changelog']}" changelog: str = (f"{self.repos.repositories[repo]['mirror_changelog']}"
f"{self.repos.repositories[repo]['changelog_txt']}") f"{self.repos.repositories[repo]['changelog_txt']}")
@ -85,11 +85,11 @@ class UpdateRepositories(Configs):
def update_slackbuild_repos(self, repo: str) -> None: def update_slackbuild_repos(self, repo: str) -> None:
""" Updates the slackbuild repositories. """ """ Updates the slackbuild repositories. """
self.utils.create_directory(self.repos.repositories[repo]['path']) # type: ignore[arg-type] self.utils.create_directory(self.repos.repositories[repo]['path'])
self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'], # type: ignore[arg-type] self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['slackbuilds_txt']) # type: ignore[arg-type] self.repos.repositories[repo]['slackbuilds_txt'])
self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'], # type: ignore[arg-type] self.utils.remove_file_if_exists(self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['changelog_txt']) # type: ignore[arg-type] self.repos.repositories[repo]['changelog_txt'])
lftp_command: str = (f"lftp {self.lftp_mirror_options} {self.repos.repositories[repo]['mirror_packages']} " lftp_command: str = (f"lftp {self.lftp_mirror_options} {self.repos.repositories[repo]['mirror_packages']} "
f"{self.repos.repositories[repo]['path']}") f"{self.repos.repositories[repo]['path']}")
@ -97,9 +97,9 @@ class UpdateRepositories(Configs):
self.multi_process.process(lftp_command) self.multi_process.process(lftp_command)
# It checks if there is a SLACKBUILDS.TXT file, otherwise it's going to create one. # It checks if there is a SLACKBUILDS.TXT file, otherwise it's going to create one.
if not Path(self.repos.repositories[repo]['path'], # type: ignore[arg-type] if not Path(self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['slackbuilds_txt']).is_file(): # type: ignore[arg-type] self.repos.repositories[repo]['slackbuilds_txt']).is_file():
self.generate.slackbuild_file(self.repos.repositories[repo]['path'], # type: ignore[arg-type] self.generate.slackbuild_file(self.repos.repositories[repo]['path'],
self.repos.repositories[repo]['slackbuilds_txt']) # type: ignore[arg-type] self.repos.repositories[repo]['slackbuilds_txt'])
self.data.install_sbo_data(repo) self.data.install_sbo_data(repo)

View file

@ -41,7 +41,7 @@ class Upgrade(Configs):
if repository == self.repos.slack_repo_name: if repository == self.repos.slack_repo_name:
extra_repo: dict = {} extra_repo: dict = {}
extra_data_file: Path = Path(self.repos.repositories[self.repos.slack_extra_repo_name]['path'], # type: ignore[arg-type] extra_data_file: Path = Path(self.repos.repositories[self.repos.slack_extra_repo_name]['path'],
self.repos.data_json) self.repos.data_json)
if self.repos.repositories[self.repos.slack_extra_repo_name]['enable'] and extra_data_file.is_file(): if self.repos.repositories[self.repos.slack_extra_repo_name]['enable'] and extra_data_file.is_file():
@ -61,7 +61,7 @@ class Upgrade(Configs):
self.installed_packages.append(Path(package)) self.installed_packages.append(Path(package))
self.installed_names.append(name) self.installed_names.append(name)
else: else:
repo_tag: str = self.repos.repositories[repository]['repo_tag'] # type: ignore[assignment] repo_tag: str = self.repos.repositories[repository]['repo_tag']
self.installed_packages: list = list(self.log_packages.glob(f'*{repo_tag}')) self.installed_packages: list = list(self.log_packages.glob(f'*{repo_tag}'))
def packages(self) -> Generator: def packages(self) -> Generator:

View file

@ -1,6 +1,8 @@
#!/usr/bin/python3 #!/usr/bin/python3
# -*- coding: utf-8 -*- # -*- coding: utf-8 -*-
# mypy: disable-error-code="no-redef"
from pathlib import Path from pathlib import Path
from slpkg.configs import Configs from slpkg.configs import Configs