style fix

.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>

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,
     )

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,