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

DOC: simplify wording for rye sync after rye add #986

Merged
merged 3 commits into from
Apr 24, 2024

Conversation

willingc
Copy link
Contributor

@willingc willingc commented Apr 2, 2024

This PR improve text in the doc so a new user of rye doesn't miss the need to rye sync after adding a dependency to actually install it. I skimmed past the original text so hopefully this will help the next reader.

@santigandolfo
Copy link

Isn't sync run automatically after add or is that just for uv? Should that be mentioned?

@charliermarsh
Copy link
Member

Isn't sync run automatically after add or is that just for uv? Should that be mentioned?

Yeah, although uv isn't the default yet so I think it's better not to introduce that complexity at this point in the docs.

@charliermarsh charliermarsh added the documentation Improvements or additions to documentation label Apr 24, 2024
@charliermarsh charliermarsh merged commit 5c0da37 into astral-sh:main Apr 24, 2024
8 checks passed
@willingc willingc deleted the doc-add-sync branch April 24, 2024 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants