Skip to content

Remove redundant # noqa annotations and enable RUF100 lint rule - #53

Merged
musantro merged 1 commit into
masterfrom
codex/revisar-y-reportar-sobre-noqa
Jul 21, 2026
Merged

musantro merged 1 commit into
masterfrom
codex/revisar-y-reportar-sobre-noqa

Conversation

@musantro

Copy link
Copy Markdown
Owner

Motivation

  • Enable the RUF100 lint rule and remove locally-suppressed # noqa annotations to bring code into alignment with stricter linting.
  • Clean up small formatting inconsistencies to improve readability without changing runtime behavior.

Description

  • Add "RUF100" to the select list in pyproject.toml to enable the new ruff rule.
  • Remove several redundant # noqa comments across files including src/trimmy/__main__.py, src/trimmy/apps/desktop/control_server.py, src/trimmy/apps/desktop/main_window.py, src/trimmy/rendering/application/render_segments_use_case.py, and src/trimmy/rendering/infrastructure/ffmpeg.py.
  • Apply minor formatting and whitespace adjustments (reflowed Annotated parameters and small function/exception handling formatting) with no changes to logic or public APIs.

Testing

  • Ran the linter with the updated configuration via ruff and the codebase passed with the new RUF100 selection.
  • Ran the test suite with pytest and all tests completed successfully.

Codex Task

@musantro
musantro enabled auto-merge (rebase) July 21, 2026 17:21
@musantro
musantro merged commit d7f88f3 into master Jul 21, 2026
9 checks passed
@musantro
musantro deleted the codex/revisar-y-reportar-sobre-noqa branch July 21, 2026 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant