chore(README): install server by PyPI package #175
Conversation
Pangjiping
requested review from
Generalwin,
hittyt,
jwx0925 and
ninan-nn
as code owners
Pangjiping
force-pushed
the
docs/server-start
branch
from
ff64cae to
4382d4c
Compare
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4382d4c199
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ```bash | ||
| git clone https://github.com/alibaba/OpenSandbox.git | ||
| cd OpenSandbox | ||
| uv pip install opensandbox-server | ||
| curl -o ~/.sandbox.toml https://raw.githubusercontent.com/alibaba/OpenSandbox/main/server/example.config.toml | ||
| ``` |
There was a problem hiding this comment.
These steps tell users to install opensandbox-server from PyPI but download example.config.toml from the repository’s main branch. If the PyPI release lags or leads main, the downloaded config can include fields not supported by the installed version (or omit required fields), and Pydantic validation in server/src/config.py will reject it at startup. This creates a real setup failure for users who follow the documented flow; consider pointing to a version-matched config (tagged release) or the packaged example instead.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
example toml is already package in opensandbox-server, can be copied from dist.
3 participants
No description provided.