From 9e0b8e9fdc4646aaae828b5db07e9b72f52c2842 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Apr 10 2017 08:38:48 +0000 Subject: pep8 fixes and add a .flake8 configuration file --- diff --git a/.flake8 b/.flake8 new file mode 100644 index 0000000..711bb18 --- /dev/null +++ b/.flake8 @@ -0,0 +1,2 @@ +[flake8] +ignore = W503 diff --git a/pagure/api/issue.py b/pagure/api/issue.py index 5e13af0..5a49842 100644 --- a/pagure/api/issue.py +++ b/pagure/api/issue.py @@ -97,8 +97,8 @@ def _check_issue_access_repo_commiter(issue): issue.private and not is_repo_committer(issue.project) and ( - not api_authenticated() or - not issue.user.user == flask.g.fas_user.username + not api_authenticated() + or not issue.user.user == flask.g.fas_user.username ) ): raise pagure.exceptions.APIError( diff --git a/tests/test_pagure_flask_ui_issues.py b/tests/test_pagure_flask_ui_issues.py index dc63e2a..fd42b0a 100644 --- a/tests/test_pagure_flask_ui_issues.py +++ b/tests/test_pagure_flask_ui_issues.py @@ -537,8 +537,8 @@ class PagureFlaskIssuestests(tests.Modeltests): 'Login to comment on this ticket.' in output.data) # Not author nor admin = No take - self.assertNotIn('function take_issue(){',output.data) - self.assertNotIn('function drop_issue(){',output.data) + self.assertNotIn('function take_issue(){', output.data) + self.assertNotIn('function drop_issue(){', output.data) self.assertNotIn( '