@@ -32,7 +32,7 @@ <h1 class="title">Module <code>continuous_delivery_scripts.assert_news</code></h
3232< span > Expand source code</ span >
3333</ summary >
3434< pre > < code class ="python "> #
35- # Copyright (C) 2020-2023 Arm Limited or its affiliates and Contributors. All rights reserved.
35+ # Copyright (C) 2020-2025 Arm Limited or its affiliates and Contributors. All rights reserved.
3636# SPDX-License-Identifier: Apache-2.0
3737#
3838"""Checks if valid news files are created for changes in the project."""
@@ -41,7 +41,7 @@ <h1 class="title">Module <code>continuous_delivery_scripts.assert_news</code></h
4141import pathlib
4242import re
4343import sys
44- from typing import List, Union
44+ from typing import Union, Optional, Iterable, Any, List
4545
4646from continuous_delivery_scripts.utils.configuration import configuration, ConfigurationVariable
4747from continuous_delivery_scripts.utils.git_helpers import ProjectTempClone, LocalProjectRepository, GitWrapper
@@ -131,6 +131,12 @@ <h1 class="title">Module <code>continuous_delivery_scripts.assert_news</code></h
131131 validate_news_file(absolute_file_path)
132132
133133
134+ def _convert_to_string_iter(list: Optional[List[Any]]) -> Iterable[str]:
135+ if list is None:
136+ return []
137+ return [str(item) for item in list]
138+
139+
134140def generate_news_file(git: GitWrapper, news_dir: pathlib.Path) -> pathlib.Path:
135141 """Adds a news file if the branch corresponds to an dependency update.
136142
@@ -147,8 +153,9 @@ <h1 class="title">Module <code>continuous_delivery_scripts.assert_news</code></h
147153 if not configuration.get_value(ConfigurationVariable.AUTOGENERATE_NEWS_FILE_ON_DEPENDENCY_UPDATE):
148154 raise EnvironmentError(f"Branch {current_branch} must contain a news file.")
149155
156+ list_groups = _convert_to_string_iter(groups)
150157 message = str(configuration.get_value(ConfigurationVariable.DEPENDENCY_UPDATE_NEWS_MESSAGE)).format(
151- message=", ".join(groups )
158+ message=", ".join(list_groups )
152159 )
153160 logger.info(f"Generating a news file with content: {message}...")
154161 return create_news_file(
@@ -163,7 +170,7 @@ <h1 class="title">Module <code>continuous_delivery_scripts.assert_news</code></h
163170 logger.info(f"Committing news file {str(news_file)}...")
164171 if not local:
165172 git.configure_for_github()
166- git.add(str( news_file) )
173+ git.add(news_file)
167174 git.commit("📰 Automatic changes ⚙ Adding news file")
168175 if not local:
169176 git.push()
@@ -289,8 +296,9 @@ <h2 id="args">Args</h2>
289296 if not configuration.get_value(ConfigurationVariable.AUTOGENERATE_NEWS_FILE_ON_DEPENDENCY_UPDATE):
290297 raise EnvironmentError(f"Branch {current_branch} must contain a news file.")
291298
299+ list_groups = _convert_to_string_iter(groups)
292300 message = str(configuration.get_value(ConfigurationVariable.DEPENDENCY_UPDATE_NEWS_MESSAGE)).format(
293- message=", ".join(groups )
301+ message=", ".join(list_groups )
294302 )
295303 logger.info(f"Generating a news file with content: {message}...")
296304 return create_news_file(
0 commit comments