diff --git a/disk_objectstore/backup_utils.py b/disk_objectstore/backup_utils.py index 67fe9c0..06c7fd4 100644 --- a/disk_objectstore/backup_utils.py +++ b/disk_objectstore/backup_utils.py @@ -82,11 +82,7 @@ def check_path_exists(self, path: Path) -> bool: return self.run_cmd(cmd)[0] def validate(self): - """Validate the backup config inputs - - :return: - True if validation passes, False otherwise. - """ + """Validate the backup config inputs""" if self.keep < 0: raise BackupError( "Input validation failed: keep variable can't be negative!" @@ -151,8 +147,6 @@ def call_rsync( # pylint: disable=too-many-arguments Add a trailing slash to the source path. This makes rsync copy the contents of the folder instead of the folder itself. - :return: - True if successful and False if unsuccessful. """ assert "rsync" in self.exes @@ -238,7 +232,7 @@ def delete_old_backups(self): else: self.logger.warning("Warning: couldn't delete old backup: %s", folder) - def backup_auto_folders(self, backup_func: Callable): + def backup_auto_folders(self, backup_func: Callable) -> None: """Create a backup, managing live and previous backup folders automatically The running backup is done to `/live-backup`. When it completes, it is moved to @@ -250,8 +244,6 @@ def backup_auto_folders(self, backup_func: Callable): Function that is used to make a single backup. Needs to have two arguments: path and previous_backup location (which can be None). - :return: - True is successful and False if unsuccessful. """ live_folder = self.path / "live-backup" @@ -331,8 +323,6 @@ def backup_container( 2) sqlite database; 3) packed files. - :return: - True if successful and False if unsuccessful. """ container_root_path = container.get_folder()