fix(build): Adjust formatting for better readability in coverage and trial build scripts

Author: hunsche

infra/build/functions/build_and_run_coverage.py

@@ -90,7 +90,7 @@ def get_build_steps(  # pylint: disable=too-many-locals, too-many-arguments
 
   if project.fuzzing_language not in LANGUAGES_WITH_COVERAGE_SUPPORT:
     return [], (f'Project "{project.name}" is written in '
-                 f'"{project.fuzzing_language}", coverage is not supported yet.')
+                f'"{project.fuzzing_language}", coverage is not supported yet.')
 
   report_date = build_project.get_datetime_now().strftime('%Y%m%d')
   bucket = CoverageBucket(project.name, report_date, PLATFORM, config.testing)
@@ -258,7 +258,7 @@ def get_build_steps(  # pylint: disable=too-many-locals, too-many-arguments
 
   build_steps.append(
       build_lib.http_upload_step(latest_report_info_body,
-                                  latest_report_info_url,
+                                 latest_report_info_url,
                                  LATEST_REPORT_INFO_CONTENT_TYPE))
   return build_steps, None
 
@@ -273,8 +273,8 @@ def get_fuzz_introspector_steps(  # pylint: disable=too-many-locals, too-many-ar
 
   if project.fuzzing_language not in LANGUAGES_WITH_INTROSPECTOR_SUPPORT:
     return [], (f'Project "{project.name}" is written in '
-                 f'"{project.fuzzing_language}", Fuzz Introspector is not '
-                 'supported yet.')
+                f'"{project.fuzzing_language}", Fuzz Introspector is not '
+                'supported yet.')
 
   build_steps = []
   build = build_project.Build(FUZZING_ENGINE, 'introspector', ARCHITECTURE)

infra/build/functions/trial_build.py

@@ -287,7 +287,6 @@ def _do_test_builds(args, test_image_suffix, end_time, version_tag):
   build_ids = collections.defaultdict(list)
   skipped_projects = []
   failed_to_start_builds = []
-  projects_to_build_count = 0
   credentials = oauth2client.client.GoogleCredentials.get_application_default()
 
   logging.info(
@@ -332,10 +331,8 @@ def _do_test_builds(args, test_image_suffix, end_time, version_tag):
     failed_to_start_builds.extend(new_failed_to_start)
     for project, project_build_id in project_builds.items():
       build_ids[project].append(project_build_id)
-            projects_to_build_count += 1
 
   logging.info('Triggered all builds.')
-
   if skipped_projects:
     logging.info(
         '================================================================')
@@ -413,7 +410,7 @@ def _do_build_type_builds(args, config, credentials, build_type, projects):
       continue
 
     steps, reason = build_type.get_build_steps_func(project_name, project_yaml,
-                                                  dockerfile_contents, config)
+                                                    dockerfile_contents, config)
     if reason:
       skipped_projects.append((project_name, reason))
       continue