Refactor: drop redundant list() wrapper around sorted() in formatter.py/text.py
Maintainer antworten meist innerhalb von 2 Tagen
Bewertung
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Anfängerfreundlichkeit
- 90/100
Rechercherichtung
Öffne awscli/formatter.py bei TableFormatter._group_scalar_keys_from_list und awscli/text.py bei _all_scalar_keys. Bestätige die drei angegebenen sorted()-Aufrufe und entferne nur deren redundante list()-Wrapper; erledigt ist dies, wenn der Refactor auf diese Stellen beschränkt bleibt und keine Verhaltensänderung eintritt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 17.4k
- Forks
- 4.8k
- Ø Merge
- 4 T. 3 Std.
- Gemergte PRs (30 T.)
- 64
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus aws/aws-cli
-
bug needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 2 Tagen
-
feature-request login p2
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
aws/aws-cli#10739 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 2 Tagen
-
aws history show/list: UnboundLocalError instead of clear error when pager fails to startEvtl. vergeben @Adityaj0 hat das vor 86 Tagen übernommen. Offenneeds-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 2 Tagen
-
CodeArtifact login: auth token leaked unredacted in CommandFailedError via subprocess stderrEvtl. vergeben @PANYAMHEMANTH hat das vor 7 Tagen übernommen. Offencodeartifact login investigating p2
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 2 Tagen
-
aws configure set s3.* can raise UnboundLocalError when updating nested config blocksEvtl. vergeben @fallintoplace hat das vor 171 Tagen übernommen. Offenbug configure p3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
aws/aws-cli#10348 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
Ähnliche Issues
-
needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
gke-labs/kube-agents#5671 ·
Maintainer antworten meist innerhalb von 1 Tag
-
docs(auth): Method B step 4 should tell users to tick "Use this client for an AI-powered agent"Offendocumentation next-release v0.9.6
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
khuisman/mcp-gee-sweet#1446 ·
Maintainer antworten meist innerhalb von 1 Tag
-
model:sonnet runtime:claude security size:S
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
FootprintAI/Containarium#3671 ·
Maintainer antworten meist innerhalb von 1 Tag
-
HolidayBase: reassigning `language` does not re-initialize translations or re-populate holidaysEvtl. vergeben @shikhabhadoria hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
vacanza/holidays#4116 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug CLI exec
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
Maintainer antworten meist innerhalb von 1 Tag