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

Ouverte Adaptée aux débutants
#10,568 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

@Adityaj0 y travaille déjà.

Depuis le 16/8/2026.

  • #10569 par @Adityaj0 — ouverte

Évaluation

Difficulté
1/5
Temps estimé
Moins d'une heure
Accessibilité débutants
90/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
python
Domaine
cli

Piste de recherche

Ouvrez awscli/formatter.py à TableFormatter._group_scalar_keys_from_list et awscli/text.py à _all_scalar_keys. Confirmez les trois appels spécifiés à sorted() et supprimez uniquement leurs wrappers redondants list() ; le travail est terminé lorsque le refactor est limité à ces emplacements, sans changement de comportement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Python
Étoiles
17.4k
Forks
4.8k
Merge moyen
4 j 3 h
PR mergées (30 j)
64

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de aws/aws-cli

Toutes les issues de aws/aws-cli

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.