diff --git a/pagure/hooks/default.py b/pagure/hooks/default.py index fc35c72..5ea0fa9 100644 --- a/pagure/hooks/default.py +++ b/pagure/hooks/default.py @@ -296,8 +296,9 @@ class DefaultRunner(BaseRunner): but_uids=pr_uids, ) - if not project.is_on_repospanner and \ - _config.get("GIT_GARBAGE_COLLECT", False): + if not project.is_on_repospanner and _config.get( + "GIT_GARBAGE_COLLECT", False + ): pagure.lib.tasks.git_garbage_collect.delay( project.repopath("main") ) diff --git a/pagure/lib/git.py b/pagure/lib/git.py index 31b5824..d477666 100644 --- a/pagure/lib/git.py +++ b/pagure/lib/git.py @@ -896,8 +896,8 @@ class TemporaryClone(object): self.repopath = tempfile.mkdtemp(prefix="pagure-%s-" % self._action) if not self._project.is_on_repospanner: # This is the simple case. Just do a local clone - # use either the specified path or the use the path of the specified - # project + # use either the specified path or the use the path of the + # specified project self._origpath = self._path or self._project.repopath( self._repotype ) @@ -1563,7 +1563,9 @@ def merge_pull_request(session, request, username, domerge=True): return "Changes merged!" else: - _log.info(" PR can be merged using fast-forward, reporting it") + _log.info( + " PR can be merged using fast-forward, reporting it" + ) request.merge_status = "FFORWARD" session.commit() return "FFORWARD" @@ -1674,7 +1676,9 @@ def merge_pull_request(session, request, username, domerge=True): pull_request=request, ) else: - _log.info(" PR can be merged using fast-forward, reporting it") + _log.info( + " PR can be merged using fast-forward, reporting it" + ) request.merge_status = "FFORWARD" session.commit() return "FFORWARD" @@ -2111,8 +2115,9 @@ def diff_pull_request( topic="pull-request.%s" % pr_action, msg=dict( pullrequest=request.to_json( - with_comments=False, public=True), - agent='pagure', + with_comments=False, public=True + ), + agent="pagure", ), ) pagure.lib.query.add_pull_request_comment( diff --git a/pagure/lib/tasks.py b/pagure/lib/tasks.py index df5b31e..eb44613 100644 --- a/pagure/lib/tasks.py +++ b/pagure/lib/tasks.py @@ -1173,7 +1173,4 @@ def git_garbage_collect(self, session, repopath): # libgit2 doesn't support "git gc" and probably never will: # https://github.com/libgit2/libgit2/issues/3247 _log.info("Running 'git gc --auto' for repo %s", repopath) - subprocess.check_output( - ["git", "gc", "--auto", "-q"], - cwd=repopath, - ) + subprocess.check_output(["git", "gc", "--auto", "-q"], cwd=repopath) diff --git a/pagure/ui/issues.py b/pagure/ui/issues.py index 9bc7d78..121bb7d 100644 --- a/pagure/ui/issues.py +++ b/pagure/ui/issues.py @@ -877,7 +877,7 @@ def new_issue(repo, username=None, namespace=None): open_access = repo.settings.get("open_metadata_access_to_all", False) milestones = [] - for m in (repo.milestones_keys or repo.milestones): + for m in repo.milestones_keys or repo.milestones: if m in repo.milestones and repo.milestones[m]["active"]: milestones.append(m) @@ -1080,7 +1080,7 @@ def view_issue(repo, issueid, username=None, namespace=None): status = pagure.lib.query.get_issue_statuses(flask.g.session) milestones = [] - for m in (repo.milestones_keys or repo.milestones): + for m in repo.milestones_keys or repo.milestones: if m in repo.milestones and repo.milestones[m]["active"]: milestones.append(m) @@ -1470,7 +1470,7 @@ def view_issue_raw_file(repo, filename=None, username=None, namespace=None): select="issues", repo=repo, username=username, - diff=data.decode('utf-8'), + diff=data.decode("utf-8"), patchfile=orig_filename, )