Skip to content

chore(deps,tooling): update ruff rules, bump ruff version#2210

Merged
danceratopz merged 2 commits intomainfrom
chore/ruff-rule-update
Sep 29, 2025
Merged

chore(deps,tooling): update ruff rules, bump ruff version#2210
danceratopz merged 2 commits intomainfrom
chore/ruff-rule-update

Conversation

@danceratopz
Copy link
Member

@danceratopz danceratopz commented Sep 26, 2025

🗒️ Description

This PR:

  1. Updates EEST's ruff rules to match those in execution-specs as closely as possible. Only 3&1/2 remaining ARG00* checks need to be enabled (and line-length 79) for parity. Issues linked below.
  2. Bumps ruff from 0.11.8 to its latest release 0.13.2.

🔗 Related Issues or PRs

They were mainly thrashed out during this PR:

which was summarized here:

Remaining tasks to obtain parity:

✅ 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. skipped
  • All: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).

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

@gurukamath gurukamath left a comment

Choose a reason for hiding this comment

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

Apart from the ingored ARG space rules, it looks good to me. But I assume the ARG stuff is going to be handled in a different PR.

@danceratopz
Copy link
Member Author

@gurukamath Yes, we'll PR those separately 👍

@danceratopz danceratopz force-pushed the chore/ruff-rule-update branch from 3d8e8cb to 4311212 Compare September 29, 2025 07:38
@danceratopz danceratopz changed the title chore(tooling): update ruff rules to more closely match ethereum/execution-specs chore(deps,tooling): update ruff rules, bump ruff version Sep 29, 2025
@danceratopz danceratopz marked this pull request as ready for review September 29, 2025 07:40
@danceratopz
Copy link
Member Author

@felix314159 I also just bumped ruff to the latest release in this PR, here's the corresponding EELS PR:

@danceratopz danceratopz merged commit 9821d2e into main Sep 29, 2025
15 checks passed
@danceratopz danceratopz deleted the chore/ruff-rule-update branch September 29, 2025 09:11
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.

3 participants