-
Notifications
You must be signed in to change notification settings - Fork 1.1k
chore: use dependency-groups for dev #2287
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
Conversation
d0e328b
to
5135b09
Compare
wow, TIL |
5135b09
to
fc37ca7
Compare
Need #2289 for pre-commit to be happy! |
The previous `[project.optional-dependencies]` was misrepresenting what the packages were. They were NOT optional dependencies to the project but development dependencies. Unlike optional dependencies, development dependencies are local-only and will not be included in the project requirements when published to PyPI or other indexes. As such, development dependencies are not included in the [project] table. Additionally, the dev group is synced by default. Source: https://docs.astral.sh/uv/concepts/projects/dependencies/#development-dependencies Signed-off-by: Sébastien Han <[email protected]>
fc37ca7
to
49f5d33
Compare
@@ -61,6 +61,7 @@ repos: | |||
"--frozen", | |||
"--no-hashes", | |||
"--no-emit-project", | |||
"--no-default-groups", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if this is right, not that I care too much about requirements.txt :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, this will actually not sync the dev
group which is synced by default.
Followup of meta-llama#2287. We must use `--group` when running commands with uv. Signed-off-by: Sébastien Han <[email protected]>
# What does this PR do? Followup of #2287. We must use `--group` when running commands with uv. <!-- Provide a short summary of what this PR does and why. Link to relevant issues if applicable. --> <!-- If resolving an issue, uncomment and update the line below --> <!-- Closes #[issue-number] --> Signed-off-by: Sébastien Han <[email protected]>
What does this PR do?
The previous
[project.optional-dependencies]
was misrepresenting what the packages were. They were NOT optional dependencies to the project but development dependencies. Unlike optional dependencies, development dependencies are local-only and will not be included in the project requirements when published to PyPI or other indexes. As such, development dependencies are not included in the [project] table. Additionally, the dev group is synced by default.Source:
https://docs.astral.sh/uv/concepts/projects/dependencies/#development-dependencies