Skip to content

Commit 3a39657

Browse files
committed
fix: remove comments and add annotation type
1 parent e26ced5 commit 3a39657

File tree

1 file changed

+1
-30
lines changed

1 file changed

+1
-30
lines changed

podman_compose.py

Lines changed: 1 addition & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -3204,25 +3204,19 @@ async def compose_up(compose: PodmanCompose, args: argparse.Namespace) -> int |
32043204

32053205
max_service_length = 0
32063206
for cnt in compose.containers:
3207-
# Saltar contenedores excluidos
32083207
if cnt["_service"] in excluded:
32093208
continue
32103209

32113210
service_name = cnt["_service"]
32123211
container_name = cnt["name"]
32133212

32143213
if getattr(args, 'names', False):
3215-
# Con -n: mostrar solo servicio_numero (sin prefijo de proyecto)
32163214
expected_name = compose.format_name(service_name, str(cnt["num"]))
3217-
32183215
if container_name == expected_name:
3219-
# Es un nombre generado automáticamente, mostrar solo servicio_numero
32203216
display_name = compose.join_name_parts(service_name, str(cnt["num"]))
32213217
else:
3222-
# Es un container_name personalizado, usarlo tal como está
32233218
display_name = container_name
32243219
else:
3225-
# Sin -n: mostrar nombre completo del contenedor (comportamiento por defecto)
32263220
display_name = container_name
32273221

32283222
curr_length = len(display_name)
@@ -3247,30 +3241,21 @@ async def handle_sigint() -> None:
32473241
loop.add_signal_handler(signal.SIGINT, lambda: asyncio.create_task(handle_sigint()))
32483242

32493243
for i, cnt in enumerate(compose.containers):
3250-
# Add colored service prefix to output like docker-compose
32513244
color_idx = i % len(compose.console_colors)
32523245
color = compose.console_colors[color_idx]
32533246

3254-
# Determinar el nombre a mostrar
32553247
service_name = cnt["_service"]
32563248
container_name = cnt["name"]
32573249

32583250
if getattr(args, 'names', False):
3259-
# Con -n: mostrar solo servicio_numero (sin prefijo de proyecto)
32603251
expected_name = compose.format_name(service_name, str(cnt["num"]))
3261-
32623252
if container_name == expected_name:
3263-
# Es un nombre generado automáticamente, mostrar solo servicio_numero
32643253
display_name = compose.join_name_parts(service_name, str(cnt["num"]))
32653254
else:
3266-
# Es un container_name personalizado, usarlo tal como está
32673255
display_name = container_name
32683256
else:
3269-
# Sin -n: mostrar nombre completo del contenedor (comportamiento por defecto)
32703257
display_name = container_name
32713258

3272-
# Calcular espacios para alinear el | exactamente
3273-
# max_service_length + 1 espacio, menos la longitud del display_name actual
32743259
space_suffix = " " * (max_service_length + 1 - len(display_name))
32753260
log_formatter = "{}{}{}|\x1b[0m".format(color, display_name, space_suffix)
32763261

@@ -3646,25 +3631,20 @@ async def compose_logs(compose: PodmanCompose, args: argparse.Namespace) -> None
36463631
)
36473632

36483633
if should_use_colors:
3649-
# Calcular la longitud máxima para alineación, igual que en compose_up
36503634
max_service_length = 0
36513635
for target in targets:
36523636
cnt = compose.container_by_name[target]
36533637
service_name = cnt["_service"]
36543638
container_name = cnt["name"]
36553639

36563640
if getattr(args, 'names', False):
3657-
# Con -n: mostrar solo servicio_numero (sin prefijo de proyecto)
36583641
expected_name = compose.format_name(service_name, str(cnt["num"]))
36593642

36603643
if container_name == expected_name:
3661-
# Es un nombre generado automáticamente, mostrar solo servicio_numero
36623644
display_name = compose.join_name_parts(service_name, str(cnt["num"]))
36633645
else:
3664-
# Es un container_name personalizado, usarlo tal como está
36653646
display_name = container_name
36663647
else:
3667-
# Sin -n: mostrar nombre completo del contenedor (comportamiento por defecto)
36683648
display_name = container_name
36693649

36703650
curr_length = len(display_name)
@@ -3673,37 +3653,28 @@ async def compose_logs(compose: PodmanCompose, args: argparse.Namespace) -> None
36733653
)
36743654

36753655
tasks = []
3676-
service_colors = {}
3656+
service_colors: dict[str, str] = {}
36773657

36783658
for target in targets:
36793659
cnt = compose.container_by_name[target]
36803660
service_name = cnt["_service"]
36813661
container_name = cnt["name"]
36823662

3683-
# Aplicar la misma lógica de display_name que en compose_up
36843663
if getattr(args, 'names', False):
3685-
# Con -n: mostrar solo servicio_numero (sin prefijo de proyecto)
36863664
expected_name = compose.format_name(service_name, str(cnt["num"]))
3687-
36883665
if container_name == expected_name:
3689-
# Es un nombre generado automáticamente, mostrar solo servicio_numero
36903666
display_name = compose.join_name_parts(service_name, str(cnt["num"]))
36913667
else:
3692-
# Es un container_name personalizado, usarlo tal como está
36933668
display_name = container_name
36943669
else:
3695-
# Sin -n: mostrar nombre completo del contenedor (comportamiento por defecto)
36963670
display_name = container_name
36973671

3698-
# Asignar color por servicio (no por contenedor individual)
36993672
if service_name not in service_colors:
37003673
color_idx = len(service_colors) % len(compose.console_colors)
37013674
service_colors[service_name] = compose.console_colors[color_idx]
37023675

37033676
color = service_colors[service_name]
37043677

3705-
# Calcular espacios para alinear el | exactamente, igual que en compose_up
3706-
# max_service_length + 1 espacio, menos la longitud del display_name actual
37073678
space_suffix = " " * (max_service_length + 1 - len(display_name))
37083679
log_formatter = "{}{}{}|\x1b[0m".format(color, display_name, space_suffix)
37093680

0 commit comments

Comments
 (0)