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

未关闭 适合新手
#10,568 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 2 天内回复

@Adityaj0 已经在做这个了。

开始于 2026年8月16日。

  • #10569 来自 @Adityaj0 —— 未关闭

评估

难度
1/5
预计耗时
1 小时以内
新手友好度
90/100
Issue 类型
重构
描述清晰度
描述清楚
活跃度
冷清
技术栈
python
领域
cli

调研方向

打开 awscli/formatter.py 中的 TableFormatter._group_scalar_keys_from_list 和 awscli/text.py 中的 _all_scalar_keys。确认指定的三个 sorted() 调用,并仅移除其冗余的 list() 包装;完成的标准是重构仅限于这些位置且不改变行为。

由索引模型根据 Issue 内容生成。

描述

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.

主要语言
Python
星标
17.4k
派生
4.8k
平均合并
4 天 3 小时
30 天内合并 PR
64

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

aws/aws-cli 的其他 Issue

查看 aws/aws-cli 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。