-
Notifications
You must be signed in to change notification settings - Fork 9
Ruff PERF401: list-comprehension findings in llm_recorder.py #488
Copy link
Copy link
Closed
Labels
good first issueAccessible entry point for new community contributorsAccessible entry point for new community contributorspriority: lowUI tweaks, documentation, and minor optimizationsUI tweaks, documentation, and minor optimizationspythonPython-specific runtime or environment optimizationsPython-specific runtime or environment optimizationsrefactorStructural cleanup or optimization without altering behaviorStructural cleanup or optimization without altering behavior
Description
Metadata
Metadata
Assignees
Labels
good first issueAccessible entry point for new community contributorsAccessible entry point for new community contributorspriority: lowUI tweaks, documentation, and minor optimizationsUI tweaks, documentation, and minor optimizationspythonPython-specific runtime or environment optimizationsPython-specific runtime or environment optimizationsrefactorStructural cleanup or optimization without altering behaviorStructural cleanup or optimization without altering behavior
Objective
Part of the ruff lint baseline cleanup (PERF401 -- a
forloop that just.append()s could be a list comprehension). Chunked out of the larger sub-issue this replaces, so it's a manageable, self-contained pickup.Findings
gitgalaxy/recorders/llm_recorder.py:466(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:513(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:606(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:619(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:691(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:725(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:747(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:1180(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:1196(PERF401) -- Uselist.extendto create a transformed listgitgalaxy/recorders/llm_recorder.py:1209(PERF401) -- Uselist.extendto create a transformed listFix in a small PR, removing resolved entries from
tests/ruff_audit_baseline.json(seetests/ruff_audit.py's module docstring for the regen command).Parent: #469