-
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
Changes from 8 commits
4b18419
c441017
2f72872
1ae6850
16ef363
098bb30
ce9fdd9
d0d4133
9adf048
a657564
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
name: Test main.py | ||
on: | ||
push: | ||
pull_request: | ||
branches: [ main ] | ||
jobs: | ||
test: | ||
runs-on: ubuntu-latest | ||
services: | ||
docker: | ||
image: docker:19.03.12 | ||
options: --privileged | ||
steps: | ||
- uses: actions/checkout@v2 | ||
rbren marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- name: Run main.py with fake model | ||
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 commentThe 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 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sounds good, will look at this when i finished #105 |
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 😅