From 5b1e07450fec0f015de629131bc4e620e07889fa Mon Sep 17 00:00:00 2001 From: Egor Tensin Date: Sun, 17 May 2026 11:52:31 +0200 Subject: reformat Python files --- plugins/action/apt_wrapper.py | 2 +- plugins/action/etckeeper_commit.py | 7 +++++-- plugins/action/safe_git.py | 12 +++++++++--- 3 files changed, 15 insertions(+), 6 deletions(-) (limited to 'plugins/action') diff --git a/plugins/action/apt_wrapper.py b/plugins/action/apt_wrapper.py index b322e10..804454b 100755 --- a/plugins/action/apt_wrapper.py +++ b/plugins/action/apt_wrapper.py @@ -22,7 +22,7 @@ class ActionModule(ActionBase): after = self._parse_dpkg_log(task_vars) - apt_result['packages'] = after[len(before):] + apt_result['packages'] = after[len(before) :] return apt_result def _parse_dpkg_log(self, task_vars, default=None): diff --git a/plugins/action/etckeeper_commit.py b/plugins/action/etckeeper_commit.py index d16e66f..06c7be0 100755 --- a/plugins/action/etckeeper_commit.py +++ b/plugins/action/etckeeper_commit.py @@ -24,11 +24,14 @@ class ActionModule(ActionBase): if allowlist: unexpected = [ - path for path in dirty_files + path + for path in dirty_files if not any([pattern.fullmatch(path) for pattern in allowlist]) ] if unexpected: - raise AnsibleActionFail(f"Unexpected modifications in repository:\n{'\n'.join(unexpected)}") + raise AnsibleActionFail( + f"Unexpected modifications in repository:\n{'\n'.join(unexpected)}" + ) return self._commit(msg, task_vars) diff --git a/plugins/action/safe_git.py b/plugins/action/safe_git.py index 7615236..8389169 100755 --- a/plugins/action/safe_git.py +++ b/plugins/action/safe_git.py @@ -34,12 +34,16 @@ class ActionModule(ActionBase): # ansible.builtin.git would just remove everything, fuck that. if not git.is_work_tree(self, dest, task_vars): - raise AnsibleActionFail(f'Destination {dest} exists but is not a repository') + raise AnsibleActionFail( + f'Destination {dest} exists but is not a repository' + ) # Again, ansible.builtin.git would just remove everything, fuck that. current_url = git.get_remote_url(self, dest, task_vars, remote=remote) if current_url != repo: - raise AnsibleActionFail(f"Existing remote URL '{current_url}' doesn't match '{repo}' for {dest}") + raise AnsibleActionFail( + f"Existing remote URL '{current_url}' doesn't match '{repo}' for {dest}" + ) # ansible.builtin.git would abort if the `force` parameter is set to # `false`, but I don't want to depend on default values. @@ -54,7 +58,9 @@ class ActionModule(ActionBase): # (again, unlike the builtin module). is_ancestor = git.is_ancestor(self, dest, remote, version, task_vars) if not is_ancestor: - raise AnsibleActionFail(f'HEAD in {dest} is not an ancestor of {remote}/{version}') + raise AnsibleActionFail( + f'HEAD in {dest} is not an ancestor of {remote}/{version}' + ) # Finally, everything looks good, delegate to the builtin module. return self._execute_module( -- cgit v1.3