Skip to content

Create E2E Tests workflow#67

Merged
ab-10 merged 44 commits intomainfrom
rml-217-rml-add-e2e-tests
Sep 10, 2025
Merged

Create E2E Tests workflow#67
ab-10 merged 44 commits intomainfrom
rml-217-rml-add-e2e-tests

Conversation

@ab-10
Copy link
Contributor

@ab-10 ab-10 commented Sep 9, 2025

This PR creates an initial E2E workflow. The workflow is incomplete but it needs to be merged into main for further testing.

Review by RecurseML

🔍 Review performed on b96f6ef..c0e566b

✨ No bugs found, your code is sparkling clean

✅ Files analyzed, no issues (2)

.github/workflows/e2e_tests.yaml
tests/e2e/test_e2e.py

⏭️ Files skipped (trigger manually) (1)
Locations Trigger Analysis
.gitignore Analyze

Analyze latest changes

Need help? Join our Discord

Copy link
Contributor

@pTinosq pTinosq left a comment

Choose a reason for hiding this comment

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

LGTM other than one minor version adjustment

- name: Install uv
uses: astral-sh/setup-uv@v5
with:
version: "0.7.2"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this not be 0.7.20 to match the version in the Makefile?

@ab-10 ab-10 merged commit 5c21236 into main Sep 10, 2025
14 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