Skip to content

Comments

Use uv to simplify the local setup instructions in the README#1246

Closed
TimChild wants to merge 1 commit intoreflex-dev:mainfrom
TimChild:simplify-setup-with-uv
Closed

Use uv to simplify the local setup instructions in the README#1246
TimChild wants to merge 1 commit intoreflex-dev:mainfrom
TimChild:simplify-setup-with-uv

Conversation

@TimChild
Copy link
Contributor

Update the Setup Locally instructions to use uv instead of venv.

I'm pretty sure you are all using uv now for package management, and it seems like a good idea to recommend to others since it's so convenient to use.

Note: the one liner uv run --with-requirements ... doesn't do hot-reloading (not sure why)... although the hot-reloading wasn't working well for me even with a full venv. I included instructions for methods of setup anyway.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Im not sure about this one do you think uv is popular enough yet for this - I want it to have what most people know already

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely that should be the aim. You might be right that it is too soon to remove python venv instructions.

I think uv has gained a lot of popularity recently given it's better support for pyproject.toml than many others, but not sure how widespread it is.

Maybe the uv one liner could just be included as well since that may be new to some uv users, and I think it's a nice way to quickly run things. But also, maybe not worth thinking about this for now.

I'll adjust the PR in the morning, but also feel free to close if you'd like.

@TimChild TimChild force-pushed the simplify-setup-with-uv branch from 420ef49 to f302a7f Compare March 27, 2025 15:56
@adhami3310
Copy link
Member

we are now using pyproject and made the instruction to do pip install .

@adhami3310 adhami3310 closed this May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants