-
Notifications
You must be signed in to change notification settings - Fork 4
Ruff misc: E402 + ambiguous-unicode grab-bag (RUF001, RUF003) #504
Copy link
Copy link
Open
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 (E402 (import not at the top of the file), RUF001 (ambiguous Unicode character in a string), RUF003 (ambiguous Unicode character in a comment)). Chunked out of the larger sub-issue this replaces, so it's a manageable, self-contained pickup.
Findings
gitgalaxy/metrics/signal_processor.py:30(E402) -- Module level import not at top of filegitgalaxy/metrics/signal_processor.py:31(E402) -- Module level import not at top of filegitgalaxy/recorders/llm_recorder.py:375(RUF001) -- String contains ambiguousℹ(INFORMATION SOURCE). Did you meani(LATIN SMALL LETTER I)?gitgalaxy/standards/language_standards.py:5899(RUF003) -- Comment contains ambiguous’(RIGHT SINGLE QUOTATION MARK). Did you mean ``` (GRAVE ACCENT)?Fix 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