#3226 Fix triggering CI checks on new comments added to a PR
Merged 6 years ago by pingou. Opened 6 years ago by pingou.

file modified
+14 -14
@@ -1294,20 +1294,20 @@ 

              'notification': notification,

          }))

  

-         # Send notification to the CI server, if the comment added was a

-         # notification and the PR is still open and project is not private

-         if notification \

-                 and request.status == 'Open' \

-                 and pagure_config.get('PAGURE_CI_SERVICES') \

-                 and request.project.ci_hook \

-                 and request.project.ci_hook.active_pr \

-                 and not request.project.private:

-             pagure.lib.tasks_services.trigger_ci_build.delay(

-                 project_name=request.project_from.fullname,

-                 cause=request.id,

-                 branch=request.branch_from,

-                 ci_type=request.project.ci_hook.ci_type

-             )

+     # Send notification to the CI server, if the comment added was a

+     # notification and the PR is still open and project is not private

+     if notification \

+             and request.status == 'Open' \

+             and pagure_config.get('PAGURE_CI_SERVICES') \

+             and request.project.ci_hook \

+             and request.project.ci_hook.active_pr \

+             and not request.project.private:

+         pagure.lib.tasks_services.trigger_ci_build.delay(

+             project_name=request.project_from.fullname,

+             cause=request.id,

+             branch=request.branch_from,

+             ci_type=request.project.ci_hook.ci_type

+         )

  

      pagure.lib.notify.log(

          request.project,

rebased onto 10cbd91

6 years ago

Pull-Request has been merged by pingou

6 years ago
Metadata