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

Improve the uv sync CLI documentation #5930

Merged
merged 2 commits into from
Aug 9, 2024
Merged

Improve the uv sync CLI documentation #5930

merged 2 commits into from
Aug 9, 2024

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Aug 8, 2024

No description provided.

@zanieb zanieb added documentation Improvements or additions to documentation cli Related to the command line interface preview Experimental behavior labels Aug 8, 2024
crates/uv-cli/src/lib.rs Outdated Show resolved Hide resolved
crates/uv-cli/src/lib.rs Outdated Show resolved Hide resolved
/// created.
///
/// The project is re-locked before syncing unless the `--locked` or
/// `--frozen` flag is provided.
Copy link
Member

Choose a reason for hiding this comment

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

"flags are"?

Copy link
Member Author

@zanieb zanieb Aug 9, 2024

Choose a reason for hiding this comment

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

Only one can be provided, idk e.g. it shortens to "unless the --locked flag is provided". Could go either way, don't know what is "correct".

///
/// Only applies to `pyproject.toml`, `setup.py`, and `setup.cfg` sources.
/// May be provided more than once.
Copy link
Member

Choose a reason for hiding this comment

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

I feel like this got removed from some other commands in previous PRs?

Copy link
Member Author

Choose a reason for hiding this comment

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

I've usually moved it out of the first line but retained it in general. I can audit again at the end.

@zanieb zanieb merged commit 921050d into main Aug 9, 2024
57 checks passed
@zanieb zanieb deleted the zb/sync-docs branch August 9, 2024 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Related to the command line interface documentation Improvements or additions to documentation preview Experimental behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants