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

Refactor I/O utils; allow 'task' command line parameter in cli.py #6187

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

enyst
Copy link
Collaborator

@enyst enyst commented Jan 10, 2025

End-user friendly description of the problem this fixes or functionality that this introduces

  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions

This PR proposes:

  • dedup CLI utils in their own utils/io.py
  • fix task argument in cli.py

Link of any specific issues this addresses
Fix #5719


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:7634d4b-nikolaik   --name openhands-app-7634d4b   docker.all-hands.dev/all-hands-ai/openhands:7634d4b

@enyst enyst requested review from xingyaoww and rbren January 10, 2025 05:23
@enyst enyst changed the title Small refactor I/O utils, task in cli mode Small refactor I/O utils, allow 'task' command line parameter in cli.py Jan 15, 2025
@enyst enyst added the lint-fix label Jan 21, 2025
@enyst enyst changed the title Small refactor I/O utils, allow 'task' command line parameter in cli.py Refactor I/O utils; allow 'task' command line parameter in cli.py Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: "-t TASK" doesn't work with CLI mode
2 participants