Skip to content

uv run does its own sync of the project env with default groups - #1746

Merged
tleonhardt merged 2 commits into
mainfrom
test-run-dependencies
Sep 6, 2026
Merged

uv run does its own sync of the project env with default groups#1746
tleonhardt merged 2 commits into
mainfrom
test-run-dependencies

Conversation

@tleonhardt

@tleonhardt tleonhardt commented Sep 6, 2026

Copy link
Copy Markdown
Member

I noticed that test were taking much longer to run on Python 3.14t and the root cause was one dependency that wasn't needed to run tests didn't have a wheel available and needed to build from source.

It turns out that uv run does its own sync of the project env with default groups.

This PR modifies all "uv run" commands for GitHub Actions workflows to only install the dependency groups they need.

This reduces the time for running these workflows, dramatically so for test jobs on free-threaded variants of Python which was the long pole in our overall CI time for PRs. Hence this speeds up the overall time for GitHub CI to run for each PR.

Tweak all "uv run" commands for GitHub Actions workflows to only install the dependency groups they need.

This reduces the time for running these workflows, dramatically so for test jobs on free-threaded variants of Python.
@tleonhardt tleonhardt self-assigned this Sep 6, 2026
@tleonhardt tleonhardt added the dependencies Pull requests that update a dependency file label Sep 6, 2026
@codecov

codecov Bot commented Sep 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.66%. Comparing base (4515353) to head (b33f2af).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1746      +/-   ##
==========================================
+ Coverage   99.64%   99.66%   +0.01%     
==========================================
  Files          23       23              
  Lines        5960     5960              
==========================================
+ Hits         5939     5940       +1     
+ Misses         21       20       -1     
Flag Coverage Δ
unittests 99.66% <ø> (+0.01%) ⬆️

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

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

This is needed for type checking because it is cmd2's only optional dependency.
@tleonhardt
tleonhardt merged commit a8b362c into main Sep 6, 2026
29 checks passed
@tleonhardt
tleonhardt deleted the test-run-dependencies branch September 6, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant