-
Notifications
You must be signed in to change notification settings - Fork 603
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
docs: Update dev docs per to describe using uv #10554
Comments
I agree, this is something that would be useful. Are you thinking about another entry in the I think you're right with the command, it might require the dev group as well. uv sync --all-extras \
--group dev \
--group tests \
--group docs |
yes, that looks like the right idea. I also want to take a stab at explaining how all the different parts of the environment are used. For example, I often just develop with uv on my host macOS, NOT inside a container, and I just run the tests with the "easy" backends like duckdb and polars, and then if they pass, then I push to a PR, and then rely on CI to actually test all the "difficult" backends for me. I think explaining this setup might be nice for devs who want a very lightweight setup. I think I will try a PR that explains this setup and see what everyone thinks. |
Just for context, I documented If you want to ensure that According to the It would be useful to document the use of I found in using the See the Lines 27 to 28 in c94db5c
Line 39 in c94db5c
|
Why don't we just make some |
Sounds good to me. Also would have better odds of staying up to date and funtional |
Typically such tools' usage are described in their own documentation. What are you trying to achieve by duplicating such information in the Ibis documentation? Am I understanding what you're after here? I think documenting, in |
Please describe the issue
The docs for setting up a dev environment don't describe how to use uv, which is how I would prefer to setup my env.
I think it is
uv sync --all-extras --group tests --group docs
? But I'm not that familiar with uv, I just started using it myself.Code of Conduct
The text was updated successfully, but these errors were encountered: