Skip to content

chore: ruff auto-fix RUF100 — unused-noqa - #31840

Closed
wesleysimplicio wants to merge 1 commit into
NousResearch:mainfrom
wesleysimplicio:chore/ruff-RUF100
Closed

chore: ruff auto-fix RUF100 — unused-noqa#31840
wesleysimplicio wants to merge 1 commit into
NousResearch:mainfrom
wesleysimplicio:chore/ruff-RUF100

Conversation

@wesleysimplicio

@wesleysimplicio wesleysimplicio commented May 25, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds the RUF100 rule to pyproject.toml and applies auto-fix across 656 files.

Removes # noqa comments that suppress lint rules no longer active in the configuration. Cleans up ~1.7k lines of stale suppressions.

Root cause

The detailed rationale from the original PR body is preserved below. This template update keeps the review structure consistent with #29640.

Fix

Why this shape

This shape mirrors #29640 so reviewers can quickly compare scope, root cause, fix, tests, and related context without having to decode a custom PR description.

Tests

  • ruff check passa com zero erros (regra já ativa na config)
  • Nenhuma mudança de comportamento em runtime — apenas transformações sintáticas
  • pytest mantém o comportamento esperado

Generated by Hermes Turbo

Related PRs / issues

  • Original body preserved below for full context.
Original body

Summary

Adds the RUF100 rule to pyproject.toml and applies auto-fix across 656 files.

Removes # noqa comments that suppress lint rules no longer active in the configuration. Cleans up ~1.7k lines of stale suppressions.

What Changed

  • pyproject.toml: added RUF100 to [tool.ruff.lint] select
  • 656 files changed: +464/-1741 lines

Fluxo

O ruff percorre a árvore do projeto aplicando a regra RUF100 onde encontra violações. As transformações foram feitas por auto-fix e mantêm o comportamento do código.

Visão

Com a regra habilitada no lint, o projeto passa a bloquear regressões futuras no mesmo padrão e fica mais consistente para novos commits.

Test Plan

  • ruff check passa com zero erros (regra já ativa na config)
  • Nenhuma mudança de comportamento em runtime — apenas transformações sintáticas
  • pytest mantém o comportamento esperado

Generated by Hermes Turbo


Generated by Hermes Turbo

@wesleysimplicio
wesleysimplicio requested a review from a team May 25, 2026 03:39
@alt-glitch

Copy link
Copy Markdown
Collaborator

@christian-byrne Tagging you on this ComfyUI item.

@alt-glitch alt-glitch added type/refactor Code restructuring, no behavior change P3 Low — cosmetic, nice to have comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery comp/plugins Plugin system and bundled plugins labels May 25, 2026
@wesleysimplicio

Copy link
Copy Markdown
Contributor Author

Closing non-fix PR as requested — mantendo apenas PRs de fix.

@wesleysimplicio
wesleysimplicio deleted the chore/ruff-RUF100 branch June 12, 2026 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have type/refactor Code restructuring, no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants