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

Include UV instructions and review installation #1459

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vcaldas
Copy link
Contributor

@vcaldas vcaldas commented Nov 9, 2024

Content of this PR:

  • Added an option to use UV
  • Clarified text
  • Add reference to the contributing page

Note: On the instructions related to topic branches, the options are bug or feature. I created this pr with the topic docreview as it is representing better the content of the PR.

Thanks for the reviews, @bug-or-feature

@kobi4000
Copy link

have you tried uv pip install -r rquirements.txt.

@vcaldas
Copy link
Contributor Author

vcaldas commented Nov 24, 2024

Yes.
Mind that these are just extra instructions and from my tests should be backwards compatible.

But happy to hear if you found issues while following what I wrote in the PR.

@kobi4000
Copy link

I encountered an issue where pip complained when I created a virtual environment with uv and tried running pip directly, displaying a message along the lines of: "This environment is externally managed..."

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.

2 participants