Refactor: drop redundant list() wrapper around sorted() in formatter.py/text.py

Abierto Apto para principiantes
#10,568 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 2 días

@Adityaj0 ya está trabajando en esto.

Desde el 16/8/2026.

  • #10569 de @Adityaj0 — abierto

Evaluación

Dificultad
1/5
Tiempo estimado
Menos de una hora
Aptitud para principiantes
90/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
python
Área
cli

Línea de trabajo

Abre awscli/formatter.py en TableFormatter._group_scalar_keys_from_list y awscli/text.py en _all_scalar_keys. Confirma las tres llamadas especificadas a sorted() y elimina únicamente sus wrappers redundantes de list(); se considera terminado cuando el refactor se limita a esas ubicaciones sin cambios de comportamiento.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

feature-request p3

Summary

`awscli/formatter.py:209-210` (`TableFormatter._group_scalar_keys_from_list`) and `awscli/text.py:91` (`_all_scalar_keys`) wrap `sorted(...)` in `list(...)`, e.g. `headers = list(sorted(headers))`. `sorted()` already returns a `list`, so the wrapper is a needless no-op call.

Proposed change

Drop the redundant `list()` wrapper in both locations: `headers = sorted(headers)`, `more = sorted(more)`, `return sorted(keys_seen)`. Pure refactor, no behavior change.

Lenguaje dominante
Python
Estrellas
17.4k
Forks
4.8k
Merge medio
4 d 3 h
PR fusionados (30 d)
64

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de aws/aws-cli

Todos los issues de aws/aws-cli

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.