Skip to content

Create E2E Tests workflow#67

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

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

Conversation

@ab-10

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

Copy link
Copy Markdown
Contributor

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

@pTinosq pTinosq left a comment

Copy link
Copy Markdown
Contributor

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

Comment thread .github/workflows/e2e_tests.yaml Outdated
- name: Install uv
uses: astral-sh/setup-uv@v5
with:
version: "0.7.2"

Copy link
Copy Markdown
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

@recurseml recurseml Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review by RecurseML

🔍 Review performed on c0e566b..f9514de

✨ No bugs found, your code is sparkling clean

✅ Files analyzed, no issues (7)

.github/workflows/e2e_tests.yaml
.pre-commit-config.yaml
pyproject.toml
src/rml/__init__.py
src/rml/package_config.py
tests/e2e/test_e2e.py
uv.lock

@recurseml recurseml Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚙️ Scanning changes in b96f6ef..f9514de for bugs...

@recurseml recurseml Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review by RecurseML

🔍 Review performed on f9514de..f9514de

✨ No files to analyze

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