-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Add basic test to make sure python runs #103
Conversation
.github/workflows/test-action.yml
Outdated
image: docker:19.03.12 | ||
options: --privileged | ||
steps: | ||
- uses: actions/checkout@v2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason why we should be using v2? It uses node 12 which is deprecated, and there are v3 and v4.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hah, the dangers of having an LLM generate code for you 😅
run: | | ||
export SANDBOX_CONTAINER_IMAGE="ubuntu:24.04" | ||
python -m pip install -r requirements.txt | ||
PYTHONPATH=`pwd` python ./opendevin/main.py -d ./ -t "write a hello world script" --model-name=fake |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xingyaoww would be good to run this a second time with the codeact agent (which I broke in an earlier PR 🙃). Would it be hard to respect the fake
model and just return a no-op?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good, will look at this when i finished #105
We have a new |
This comes out of me almost breaking main.py while editing the server.