Update contribution document #327
Merged
+26
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
seanGammage
changed the title
ProKil
reviewed
Oct 30, 2025
| ### Without Dev Containers | ||
| If you do not want to use containers, you can run everything directly on your machine. | ||
| 1. You'll need to ensure you have your pyproject.toml and / or requirments.txt files | ||
| 1. You'll need `poetry` or `pip` to install the project dependencies. |
Member
There was a problem hiding this comment.
We don't use poetry but use uv now
There was a problem hiding this comment.
I'm on it! XuhuiZhou can track my progress at all-hands.dev
There was a problem hiding this comment.
Summary
I have successfully addressed @ProKil's comment on PR #327 requesting to update the documentation from using poetry to uv.
Key Changes Made:
- Updated dependency management: Replaced
poetryreference withuvin the "Without Dev Containers" section (line 170) - Modernized installation instructions: Changed from outdated
pip install -r requirements.txtapproach to currentuv sync --all-extrasworkflow - Fixed typos and grammar: Corrected "accross" → "across", "IDE's" → "IDEs", "build" → "built", "less" → "fewer"
- Improved guidance: Enhanced "Other IDEs" section with proper Dev Containers documentation reference
Technical Execution:
- Successfully pushed changes directly to the PR author's branch (
seanGammage/sotopia:add-dev-container-instructions) - PR Update contribution document #327 has been updated with commit
28fcab19e962962bc704e0aab621837f26a2d367 - Changes are now live and address the maintainer's feedback
The contribution documentation now correctly reflects that Sotopia uses uv for dependency management, making it consistent with the project's current tooling and addressing @ProKil's comment completely.
- Replace poetry/pip references with uv in the 'Without Dev Containers' section - Fix typos: 'accross' -> 'across', 'IDE's' -> 'IDEs', 'build' -> 'built', 'less' -> 'fewer' - Improve instructions for other IDEs to reference Dev Containers documentation - Make dependency installation consistent with existing uv workflow Co-authored-by: openhands <openhands@all-hands.dev>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This MR:
Note
Adds development instructions for using Dev Containers (VS Code and other IDEs) and a local setup path without containers using uv.
docs/pages/contribution/contribution.mdwith a new section: Using Dev Containers for development.uv, and syncing dependencies withuv sync --all-extras.Written by Cursor Bugbot for commit 28fcab1. This will update automatically on new commits. Configure here.