Skip to content

Commit ad760fd

Browse files
committed
Remove _get_relative_meta_name
Current implementation does not bring much benefit and can be replaced with a simple f-string. Signed-off-by: Teodora Sechkova <[email protected]>
1 parent e2db9d2 commit ad760fd

File tree

1 file changed

+1
-15
lines changed

1 file changed

+1
-15
lines changed

tuf/client_rework/updater_rework.py

+1-15
Original file line numberDiff line numberDiff line change
@@ -191,20 +191,6 @@ def _get_full_meta_name(
191191
filename,
192192
)
193193

194-
@staticmethod
195-
def _get_relative_meta_name(
196-
role: str, extension: str = ".json", version: int = None
197-
) -> str:
198-
"""
199-
Helper method returning full metadata file path given the role name
200-
and file extension.
201-
"""
202-
if version is None:
203-
filename = role + extension
204-
else:
205-
filename = str(version) + "." + role + extension
206-
return filename
207-
208194
def _load_root(self) -> None:
209195
"""
210196
If metadata file for 'root' role does not exist locally, download it
@@ -228,7 +214,7 @@ def _load_root(self) -> None:
228214
try:
229215
root_mirrors = mirrors.get_list_of_mirrors(
230216
"meta",
231-
self._get_relative_meta_name("root", version=next_version),
217+
f"{next_version}.root.json",
232218
self._mirrors,
233219
)
234220

0 commit comments

Comments
 (0)