MethodicConfigurator icon indicating copy to clipboard operation
MethodicConfigurator copied to clipboard

Use uv to install pytest environment

Open amilcarlucas opened this issue 6 months ago • 1 comments

This pull request updates the workflow configuration in .github/workflows/pytest.yml to integrate with the uv tool for managing Python environments, replacing the previous actions/setup-python step. This change simplifies environment setup and dependency installation.

Workflow updates for Python environment setup:

  • Replaced actions/setup-python with astral-sh/setup-uv to manage Python environments, enabling the activate-environment option for seamless integration.
  • Updated the dependency installation process to use uv pip install instead of directly invoking pip, aligning with the uv tool's workflow.

amilcarlucas avatar May 05 '25 12:05 amilcarlucas

Test Results

    2 files  ±0      2 suites  ±0   1m 21s ⏱️ -1s 1 565 tests ±0  1 564 ✅ ±0  1 💤 ±0  0 ❌ ±0  3 130 runs  ±0  3 128 ✅ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 57d2605c. ± Comparison against base commit 388216c1.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar May 05 '25 12:05 github-actions[bot]

pip works fine here and all the tests do pass.

But I wanted to speed it up. uv is one full minute faster, so I wanted to switch to it. But uv has issues, the tests no longer pass, it crashes at some GUI test :( I tried to disable pip installation to see if that fixes the failing test, but it does not :(

amilcarlucas avatar Jun 17 '25 12:06 amilcarlucas

uv v0.8... was released in the past 24 hours.

  • https://github.com/astral-sh/uv/releases

cclauss avatar Jul 18 '25 11:07 cclauss

I just updated it to 6.4.1

amilcarlucas avatar Jul 18 '25 12:07 amilcarlucas

I hope to have fixed this with commit 9469e1b9fdab98b92aa72cc49591bf7bbd70bb7e

amilcarlucas avatar Jul 19 '25 17:07 amilcarlucas

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
7873 6058 77% 73% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 57d2605 by action🐍

github-actions[bot] avatar Aug 14 '25 12:08 github-actions[bot]

Looks like UV fixed their act! This can finally get merged! It saves time when running the tests as it installs the dependencies faster

amilcarlucas avatar Aug 17 '25 17:08 amilcarlucas