Skip to content

Commit

Permalink
Merge branch 'ci/delete_preserve_param' into 'master'
Browse files Browse the repository at this point in the history
ci: Delete preserve param in build_apps.py

See merge request adf/esp-adf-internal!1343
  • Loading branch information
jason-mao committed Sep 27, 2024
2 parents 1bc914b + 4989260 commit 14c79a0
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tools/ci/build_apps.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ def get_cmake_apps(
build_log_filename='build_log.txt',
size_json_filename='size.json',
check_warnings=True,
preserve=True,
default_build_targets=default_build_targets,
manifest_files=[str(p) for p in Path(os.environ['ADF_PATH']).glob('**/.build-test-rules.yml')],
)
Expand All @@ -106,7 +105,7 @@ def main(args): # type: (argparse.Namespace) -> None
else:
apps_to_build = apps[:]

print('Found %d apps after filtering', len(apps_to_build))
print(f'Found {len(apps_to_build)} apps after filtering')
print(
'Suggest setting the parallel count to %d for this build job',
len(apps_to_build) // APPS_BUILD_PER_JOB + 1,
Expand Down

0 comments on commit 14c79a0

Please sign in to comment.