From f1421c9e5c542ed529dd3b225fc06c696a054080 Mon Sep 17 00:00:00 2001
From: Max K <maxkaynov@gmail.com>
Date: Fri, 24 May 2024 12:02:14 +0200
Subject: [PATCH] style fix

---
 .github/PULL_REQUEST_TEMPLATE.md | 11 +++++------
 tests/ci/commit_status_helper.py |  4 +---
 tests/ci/finish_check.py         |  4 ++--
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index 663b464d002..f9765c1d57b 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -46,9 +46,8 @@ At a minimum, the following information should be added (but add more as needed)
 
 **NOTE:** If your merge the PR with modified CI you **MUST KNOW** what you are doing
 **NOTE:** Checked options will be applied if set before CI RunConfig/PrepareRunConfig step
----
 - [ ] <!---ci_include_integration--> Allow: Integration Tests
-- [ ] <!---ci_include_stateless--> Allow:: Stateless tests
+- [ ] <!---ci_include_stateless--> Allow: Stateless tests
 - [ ] <!---ci_include_stateful--> Allow: Stateful tests
 - [ ] <!---ci_include_unit--> Allow: Unit tests
 - [ ] <!---ci_include_performance--> Allow: Performance tests
@@ -74,8 +73,8 @@ At a minimum, the following information should be added (but add more as needed)
 - [ ] <!---do_not_test--> do not test (only style check)
 - [ ] <!---no_merge_commit--> disable merge-commit (no merge from master before tests)
 - [ ] <!---no_ci_cache--> disable CI cache (job reuse)
-- [ ] <!---batch_0--> only batch 1 for multi-batch jobs
-- [ ] <!---batch_1--> only batch 2 for multi-batch jobs
-- [ ] <!---batch_2--> only batch 3 for multi-batch jobs
-- [ ] <!---batch_3--> only batch 4 for multi-batch jobs
+- [ ] <!---batch_0--> allow: batch 1 for multi-batch jobs
+- [ ] <!---batch_1--> allow: batch 2
+- [ ] <!---batch_2--> allow: batch 3
+- [ ] <!---batch_3_4_5--> allow: batch 4, 5 and 6
 </details>
diff --git a/tests/ci/commit_status_helper.py b/tests/ci/commit_status_helper.py
index 22cc0085781..bdbb0e80653 100644
--- a/tests/ci/commit_status_helper.py
+++ b/tests/ci/commit_status_helper.py
@@ -20,7 +20,6 @@ from github.Repository import Repository
 from ci_config import CHECK_DESCRIPTIONS, CheckDescription, StatusNames, is_required
 from env_helper import (
     GITHUB_REPOSITORY,
-    GITHUB_RUN_URL,
     GITHUB_UPSTREAM_REPOSITORY,
     TEMP_PATH,
 )
@@ -557,13 +556,12 @@ def update_upstream_sync_status(
     post_commit_status(
         last_synced_upstream_commit,
         sync_status,
-        "",  # let's won't expose any urls from cloud
+        "",
         "",
         StatusNames.SYNC,
     )
     trigger_mergeable_check(
         last_synced_upstream_commit,
         get_commit_filtered_statuses(last_synced_upstream_commit),
-        True,
         set_if_green=can_set_green_mergeable_status,
     )
diff --git a/tests/ci/finish_check.py b/tests/ci/finish_check.py
index 130973ee8ff..269d5aa3175 100644
--- a/tests/ci/finish_check.py
+++ b/tests/ci/finish_check.py
@@ -15,7 +15,7 @@ from commit_status_helper import (
 )
 from get_robot_token import get_best_robot_token
 from pr_info import PRInfo
-from report import PENDING, SUCCESS
+from report import PENDING
 from synchronizer_utils import SYNC_BRANCH_PREFIX
 from env_helper import GITHUB_REPOSITORY, GITHUB_UPSTREAM_REPOSITORY
 
@@ -67,7 +67,7 @@ def main():
         if status.state == PENDING:
             post_commit_status(
                 commit,
-                state, # map Mergeable Check status to CI Running
+                state,  # map Mergeable Check status to CI Running
                 status.target_url,
                 "All checks finished",
                 StatusNames.CI,