chore(deps,tooling): move `dev` extras to `dependency-group` by danceratopz · Pull Request #1688 · ethereum/execution-specs

danceratopz · 2025-10-27T10:41:29Z

🗒️ Description

tldr

Fix testing package by adding ruff as a dep (required by gentest).
Move developer dependencies from "extras" (optional dependencies) to PEP 753 "dependency groups".

What changes?

Developers need to run:

uv sync

to get all dev dependencies (previously it was uv sync --all-extras).

More Background

These changes follow a retroactive review of #1671, sorry, I was too slow @fselmo!

I think the standard way of adding dev dependencies is not as an "extra", but rather as a dependency-group:

uv reads development dependencies from the [dependency-groups] table (as defined in PEP 735).

The dev group is special-cased and synced by default.
From https://docs.astral.sh/uv/concepts/projects/sync/#syncing-development-dependencies

So this PR makes that change and additionally removes tox as a direct dev dependency; imo, it should be ran as a "tool" via uvx; the doc changes already made in

chore: remove unnecessary flag from tox commands #1668

reflect that.

Before this PR:

uv sync --extras dev  # unnecessary
uv run tox -e static  # feels a bit weird

With this PR:

uv sync  # installs dev deps by default
uvx tox -e static  # treat tox as a tool

It's great that we get to drop the --with=tox-uv flag!

🔗 Related Issues or PRs

N/A.

✅ Checklist

All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
```
uvx tox -e static
```
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).

Cute Animal Picture

- `min_version` is deprecated: https://tox.wiki/en/latest/config.html#min_version

codecov-commenter · 2025-10-27T16:14:32Z

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.07%. Comparing base (abd7612) to head (bd5dbd3).
⚠️ Report is 3 commits behind head on forks/osaka.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff              @@
##           forks/osaka    #1688   +/-   ##
============================================
  Coverage        86.07%   86.07%           
============================================
  Files              743      743           
  Lines            44078    44078           
  Branches          3894     3894           
============================================
  Hits             37938    37938           
  Misses            5659     5659           
  Partials           481      481

Flag	Coverage Δ
unittests	`86.07% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Simplify tox.ini by always installing all dev dependencies.

danceratopz · 2025-10-28T11:21:54Z

@fselmo @SamWilsn let me know if you like it, I can make a follow-up to update the docs.

danceratopz

I fixed up a packaging bug in testing that cropped up due to these changes; it was missing ruff as a dependency.

This reverts commit 661a1b5. This will be handled differently: ethereum#1715

gentest still requires ruff, cf ethereum#1715

fselmo

lgtm once CI passes 👍🏼

danceratopz · 2025-10-29T20:33:17Z

lgtm once CI passes 👍🏼

Thanks, now we're failing testing unit tests (here) due to the issue that @felix314159 is addressing here

fix(pytest): fix failing unit test test_model_copy[Environment] #1711

fselmo · 2025-10-29T21:11:48Z

Yep sounds good on CI fix in separate PR if you're good with it 👍🏼

chore(deps,tooling): move dev deps to dependency-groups; rm tox

baec4e9

danceratopz added the A-deps Area: Dependencies—Stuff we build on top of (eg. `uv.lock`, `pyproject.toml`) label Oct 27, 2025

chore(tooling): remove deprecated min_version; stricter ver specs

bd5dbd3

- `min_version` is deprecated: https://tox.wiki/en/latest/config.html#min_version

danceratopz force-pushed the chore/fix-dev-deps branch from 6539bcd to bd5dbd3 Compare October 27, 2025 10:48

SamWilsn reviewed Oct 27, 2025

View reviewed changes

Comment thread pyproject.toml Outdated

danceratopz added 2 commits October 28, 2025 11:22

chore(packaging): move "extras" to dependency-groups

e0993f2

Simplify tox.ini by always installing all dev dependencies.

chore(packaging): move testing "extras" to dependency-groups

82703a4

danceratopz commented Oct 28, 2025

View reviewed changes

Comment thread tox.ini

chore(tox): do not install optimized in pypy envs

a4c85ce

danceratopz commented Oct 28, 2025

View reviewed changes

Comment thread tox.ini

chore(packaging): add ruff to deps; gentest requires it

661a1b5

danceratopz commented Oct 28, 2025

View reviewed changes

Comment thread tox.ini

Comment thread packages/testing/pyproject.toml Outdated

SamWilsn reviewed Oct 29, 2025

View reviewed changes

Comment thread pyproject.toml

Comment thread pyproject.toml

Comment thread tox.ini

fselmo reviewed Oct 29, 2025

View reviewed changes

Comment thread tox.ini

danceratopz mentioned this pull request Oct 29, 2025

Update docs for the change from optional-dependencies (extras) to dependency_groups #1713

Closed

danceratopz added 3 commits October 29, 2025 20:38

Revert "chore(packaging): add ruff to deps; gentest requires it"

ef482a0

This reverts commit 661a1b5. This will be handled differently: ethereum#1715

chore(tox): add comment why we specify tests for pypy envs

81d126e

chore(deps): move tools dependency group deps to package deps

073779f

danceratopz force-pushed the chore/fix-dev-deps branch from 91d3c92 to 073779f Compare October 29, 2025 19:40

fix(tox): add lint dependency group for testing unit tests

38c7298

gentest still requires ruff, cf ethereum#1715

fselmo approved these changes Oct 29, 2025

View reviewed changes

danceratopz merged commit 3d638d9 into ethereum:forks/osaka Oct 30, 2025
16 of 19 checks passed

danceratopz added a commit to danceratopz/execution-specs that referenced this pull request Nov 6, 2025

docs: dev deps are no longer package extras cf ethereum#1688

a213343

SamWilsn pushed a commit that referenced this pull request Nov 7, 2025

docs: dev deps are no longer package extras cf #1688

1078413

danceratopz mentioned this pull request Jan 19, 2026

simulators/eels: fix execute-blobs simulator ethereum/hive#1365

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore(deps,tooling): move `dev` extras to `dependency-group`#1688

chore(deps,tooling): move `dev` extras to `dependency-group`#1688
danceratopz merged 10 commits into
ethereum:forks/osakafrom
danceratopz:chore/fix-dev-deps

danceratopz commented Oct 27, 2025 •

edited

Loading

Uh oh!

Uh oh!

codecov-commenter commented Oct 27, 2025

Uh oh!

Uh oh!

Uh oh!

danceratopz commented Oct 28, 2025

Uh oh!

danceratopz left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

fselmo left a comment

Uh oh!

danceratopz commented Oct 29, 2025

Uh oh!

fselmo commented Oct 29, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

danceratopz commented Oct 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🗒️ Description

tldr

What changes?

More Background

🔗 Related Issues or PRs

✅ Checklist

Cute Animal Picture

Uh oh!

Uh oh!

codecov-commenter commented Oct 27, 2025

Codecov Report

Uh oh!

Uh oh!

Uh oh!

danceratopz commented Oct 28, 2025

Uh oh!

danceratopz left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

fselmo left a comment

Choose a reason for hiding this comment

Uh oh!

danceratopz commented Oct 29, 2025

Uh oh!

fselmo commented Oct 29, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

danceratopz commented Oct 27, 2025 •

edited

Loading