Skip to content

Conversation

@franciscojavierarceo
Copy link
Member

@franciscojavierarceo franciscojavierarceo commented Feb 10, 2026

Summary

  • Fix the benchmark step in master_only.yml to use uv run pytest instead of calling pytest directly
  • This ensures pytest runs in the uv-managed virtual environment where dependencies are installed

Test plan

  • Verify CI passes on master branch push

🤖 Generated with Claude Code


Open with Devin

franciscojavierarceo and others added 2 commits February 10, 2026 11:27
The benchmark step was calling pytest directly instead of using uv run,
which would fail since dependencies are installed in the uv-managed
virtual environment.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@franciscojavierarceo franciscojavierarceo merged commit 5096010 into master Feb 10, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants