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.
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.