From 9602c600ab18062715700bb543ed84530a2f8632 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Apr 18 2017 11:52:31 +0000 Subject: flake8 fixes to pagure.api Signed-off-by: Pierre-Yves Chibon --- diff --git a/pagure/api/__init__.py b/pagure/api/__init__.py index 21eab81..6482699 100644 --- a/pagure/api/__init__.py +++ b/pagure/api/__init__.py @@ -187,14 +187,14 @@ def api_method(function): if err.error_code in [APIERROR.ENOCODE]: output = { - 'error': err.error, - 'error_code': err.error_code.name - } + 'error': err.error, + 'error_code': err.error_code.name + } else: output = { - 'error': err.error_code.value, - 'error_code': err.error_code.name, - } + 'error': err.error_code.value, + 'error_code': err.error_code.name, + } if err.errors: output['errors'] = err.errors diff --git a/pagure/api/group.py b/pagure/api/group.py index 78ab1b2..24aacea 100644 --- a/pagure/api/group.py +++ b/pagure/api/group.py @@ -91,24 +91,24 @@ def api_view_group(group): { "creator": { - "default_email": "user1@example.com", + "default_email": "user1@example.com", "emails": [ "user1@example.com" - ], - "fullname": "User1", + ], + "fullname": "User1", "name": "user1" - }, - "date_created": "1492011511", - "description": "Some Group", - "display_name": "Some Group", - "group_type": "user", + }, + "date_created": "1492011511", + "description": "Some Group", + "display_name": "Some Group", + "group_type": "user", "members": [ - "user1", + "user1", "user2" - ], + ], "name": "some_group_name" } - """ + """ # noqa group = pagure.lib.search_groups(SESSION, group_name=group) if not group: raise pagure.exceptions.APIError(404, error_code=APIERROR.ENOGROUP) diff --git a/pagure/api/issue.py b/pagure/api/issue.py index 89708fb..1497c75 100644 --- a/pagure/api/issue.py +++ b/pagure/api/issue.py @@ -18,8 +18,7 @@ import pagure.exceptions import pagure.lib from pagure import ( - APP, SESSION, is_repo_committer, api_authenticated, urlpattern, - is_repo_admin + APP, SESSION, is_repo_committer, api_authenticated, urlpattern ) from pagure.api import ( API, api_method, api_login_required, api_login_optional, APIERROR @@ -31,7 +30,8 @@ def _get_repo(repo_name, username=None, namespace=None): :param repo_name: name of repository :param username: :param namespace: - :raises pagure.exceptions.APIError: when repository doesn't exists or is disabled + :raises pagure.exceptions.APIError: when repository doesn't exists or + is disabled :return: repository name """ repo = pagure.get_authorized_project( @@ -372,7 +372,7 @@ def api_view_issues(repo, username=None, namespace=None): no_stones = True else: no_stones = False - priority = flask.request.args.get('priority', None) + priority = flask.request.args.get('priority', None) since = flask.request.args.get('since', None) status = flask.request.args.get('status', None) tags = flask.request.args.getlist('tags') @@ -667,7 +667,6 @@ def api_change_status_issue(repo, issueid, username=None, namespace=None): issue = _get_issue(repo, issueid) _check_issue_access_repo_commiter(issue) - status = pagure.lib.get_issue_statuses(SESSION) form = pagure.forms.StatusForm( status=status, @@ -724,9 +723,11 @@ def api_change_status_issue(repo, issueid, username=None, namespace=None): @API.route('//issue//milestone', methods=['POST']) -@API.route('///issue//milestone', methods=['POST']) @API.route( - '/fork///issue//milestone', methods=['POST']) + '///issue//milestone', methods=['POST']) +@API.route( + '/fork///issue//milestone', + methods=['POST']) @API.route( '/fork////issue//milestone', methods=['POST']) @@ -770,7 +771,7 @@ def api_change_milestone_issue(repo, issueid, username=None, namespace=None): "message": "Successfully edited issue #1" } - """ + """ # noqa output = {} repo = _get_repo(repo, username, namespace) _check_issue_tracker(repo) diff --git a/pagure/api/project.py b/pagure/api/project.py index cc5eb3e..797d2b8 100644 --- a/pagure/api/project.py +++ b/pagure/api/project.py @@ -165,7 +165,8 @@ def api_projects(): private = flask.g.fas_user.username projects = pagure.lib.search_projects( - SESSION, username=username, fork=fork, tags=tags, pattern=pattern, private=private) + SESSION, username=username, fork=fork, + tags=tags, pattern=pattern, private=private) if not projects: raise pagure.exceptions.APIError(