From 03a70ddb679bf40906eb7d7c324ce927e36a3483 Mon Sep 17 00:00:00 2001 From: Farhaan Bukhsh Date: Jul 28 2016 09:44:30 +0000 Subject: [PATCH 1/2] Fix Pep 8 --- diff --git a/tests/test_pagure_flask_ui_plugins.py b/tests/test_pagure_flask_ui_plugins.py index bda5f7f..2d9469d 100644 --- a/tests/test_pagure_flask_ui_plugins.py +++ b/tests/test_pagure_flask_ui_plugins.py @@ -71,8 +71,8 @@ class PagureFlaskPluginstests(tests.Modeltests): sorted(names), [ 'Block Un-Signed commits', 'Block non fast-forward pushes', - 'Fedmsg', 'IRC', 'Mail', 'Pagure', 'Pagure CI','Pagure requests', - 'Pagure tickets', 'Read the Doc', + 'Fedmsg', 'IRC', 'Mail', 'Pagure', 'Pagure CI', + 'Pagure requests', 'Pagure tickets', 'Read the Doc', ] ) From 0064dfe9fd29cb99d122210a49882bd35c4368b4 Mon Sep 17 00:00:00 2001 From: Farhaan Bukhsh Date: Jul 28 2016 15:23:02 +0000 Subject: [PATCH 2/2] Fix pep 8 and cleaning code --- diff --git a/pagure/lib/pagure_ci.py b/pagure/lib/pagure_ci.py index 643f533..88ddcd7 100644 --- a/pagure/lib/pagure_ci.py +++ b/pagure/lib/pagure_ci.py @@ -24,7 +24,7 @@ JENKINS_TRIGGER_URL = '{base}job/{project}/buildWithParameters' def process_pr(logger, cfg, pr_id, repo, branch): - ''' Function to process the PR, it POST the + ''' Process the pull-request, it POST the data to the Jenkins URL to trigger build. ''' if cfg.active: @@ -40,8 +40,8 @@ def process_pr(logger, cfg, pr_id, repo, branch): def process_build(logger, cfg, build_id): - ''' Function is used to get the build info - from jenkins and flag that particular pull-request. + ''' Gets the build info from jenkins + and flags that particular pull-request. ''' if cfg.active: jenk = jenkins.Jenkins(cfg.jenkins_url) diff --git a/pagure/templates/plugin.html b/pagure/templates/plugin.html index d98af1a..ab48797 100644 --- a/pagure/templates/plugin.html +++ b/pagure/templates/plugin.html @@ -21,7 +21,9 @@
- +
{% endif %}