Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CI triggers: Only run Python tests when Python files change #6654

Closed
wants to merge 2 commits into from

Conversation

neubig
Copy link
Contributor

@neubig neubig commented Feb 7, 2025

This PR fixes the CI configuration to only run Python tests when Python files are changed.

Currently, the Python tests are running on all PRs, even when only frontend files are changed. This causes unnecessary CI runs and potential failures due to missing frontend dependencies in the Python test environment.

Changes:

  • Added paths filter to the Python test workflow to only run when Python files or related files are changed
  • Files that trigger Python tests:
    • openhands/**
    • agenthub/**
    • evaluation/**
    • tests/**
    • pyproject.toml
    • poetry.lock
    • .github/workflows/py-unit-tests.yml

This change will make the CI more efficient and avoid false failures.


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:840ebf7-nikolaik   --name openhands-app-840ebf7   docker.all-hands.dev/all-hands-ai/openhands:840ebf7

@neubig neubig closed this Feb 7, 2025
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