Skip to content

chore(tooling): Enable all ruff ARG rules and fix all violations#2291

Merged
fselmo merged 1 commit intoethereum:mainfrom
fselmo:feat/ruff-arg-checks-parity
Oct 10, 2025
Merged

chore(tooling): Enable all ruff ARG rules and fix all violations#2291
fselmo merged 1 commit intoethereum:mainfrom
fselmo:feat/ruff-arg-checks-parity

Conversation

@fselmo
Copy link
Collaborator

@fselmo fselmo commented Oct 10, 2025

🗒️ Description

Enable all ARG ruff checks - Closes #2176

Deferred to del approach discussed here

✅ Checklist

  • All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    uvx --with=tox-uv tox -e lint,typecheck,spellcheck,markdownlint
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
  • All: Considered adding an entry to CHANGELOG.md.
  • All: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).

@fselmo fselmo added type:chore Type: Chore scope:tooling Scope: Python tools (uv, ruff, tox,...) labels Oct 10, 2025
Copy link
Collaborator

@kclowes kclowes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@fselmo fselmo force-pushed the feat/ruff-arg-checks-parity branch from 8f0814e to b6b915c Compare October 10, 2025 21:52
@fselmo fselmo force-pushed the feat/ruff-arg-checks-parity branch from b6b915c to 40bae5e Compare October 10, 2025 21:56
@fselmo fselmo merged commit d1d62bc into ethereum:main Oct 10, 2025
22 of 30 checks passed
@fselmo fselmo deleted the feat/ruff-arg-checks-parity branch October 10, 2025 22:17
kclowes pushed a commit to kclowes/execution-spec-tests that referenced this pull request Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:tooling Scope: Python tools (uv, ruff, tox,...) type:chore Type: Chore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore(tooling): enable remaining ruff ARG checks

2 participants