Skip to content

Commit 703bb51

Browse files
committed
fix(uv): Remove unnecessary --all-extras and --all-groups from sync commands
The `--all-extras` and `--all-groups` flags for `uv sync` were causing issues with dependency resolution and environment setup in local development, direnv integration, and ReadTheDocs builds. Removing these flags simplifies the `uv sync` command, relying on its default behavior which correctly handles the project's dependencies without these specific options, thus fixing potential installation and environment activation problems.
1 parent 47f75a0 commit 703bb51

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

template/.config/copier/mise-tasks/install.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,5 @@ fi
1818
if [[ -f 'uv.lock' ]]; then
1919
uv='uv'
2020
if has uv-wrapper.sh; then uv='uv-wrapper.sh'; fi
21-
"$uv" sync --all-extras --all-groups "$@"
21+
"$uv" sync "$@"
2222
fi

template/.config/direnv/10-python.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ fi
2121
if [[ -f 'uv.lock' ]]; then
2222
uv='uv'
2323
if has uv-wrapper.sh; then uv='uv-wrapper.sh'; fi
24-
"$uv" sync --all-extras --all-groups
24+
"$uv" sync
2525
sed --in-place --regexp-extended \
2626
's|\s*(include-system-site-packages)\s*=\s*.*\s*|\1 = true|' '.venv/pyvenv.cfg'
2727
# shellcheck disable=SC1091

template/.readthedocs.yaml.jinja

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ build:
3838
# %% if package_manager == "pixi":
3939
- pixi install --frozen
4040
# %% elif package_manager == "uv":
41-
- UV_PROJECT_ENVIRONMENT="${READTHEDOCS_VIRTUALENV_PATH}" uv sync --all-extras --all-groups --frozen --all-packages --link-mode copy
41+
- UV_PROJECT_ENVIRONMENT="${READTHEDOCS_VIRTUALENV_PATH}" uv sync --frozen --all-packages --link-mode copy
4242
# %% endif
4343

4444
pre_build:

0 commit comments

Comments
 (0)