Skip to content

Fix handling of UV_NO_SYNC=1 uv run ... #17391

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.

Sign up for GitHub

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

Jump to bottom
Merged
zanieb merged 2 commits into from
Jan 9, 2026
Merged

Fix handling of UV_NO_SYNC=1 uv run ... #17391

zanieb merged 2 commits into from
Jan 9, 2026
+61 −1

Conversation

@zanieb
Copy link
Member

@zanieb zanieb commented Jan 9, 2026 •
edited
Loading

Closes #17390

claude added 2 commits January 9, 2026 20:53
@claude
Add failing test for UV_NO_SYNC=1 env var not working with uv run
b001edb 
This test demonstrates issue astral-sh#17390 where setting `UV_NO_SYNC=1`
environment variable does not prevent `uv run` from syncing the
environment.

The root cause is that `RunSettings::resolve()` doesn't use
`resolve_flag()` to combine the CLI argument with the environment
variable, unlike other commands like `uv add`, `uv remove`, and
`uv version` which handle it correctly.
@claude
Fix UV_NO_SYNC=1 not working for uv run
53ecea1 
The `RunSettings::resolve()` function was passing the `no_sync` CLI
argument directly without using `resolve_flag()` to combine it with
the `UV_NO_SYNC` environment variable.

This fix adds `resolve_flag(no_sync, "no-sync", environment.no_sync)`
to properly resolve the flag from both CLI and environment variable
sources, matching how other commands (add, remove, version) already
handle this flag.

Fixes astral-sh#17390
@zanieb zanieb changed the title Add regression test for UV_NO_SYNC=1 uv run ... Fix handling of UV_NO_SYNC=1 uv run ... Jan 9, 2026
@zanieb zanieb marked this pull request as ready for review January 9, 2026 21:07
@zanieb zanieb added the bug Something isn't working label Jan 9, 2026
@zanieb zanieb requested a review from charliermarsh January 9, 2026 21:07
@zanieb zanieb enabled auto-merge (squash) January 9, 2026 21:10
@zanieb zanieb merged commit 1a72f8c into astral-sh:main Jan 9, 2026
188 of 191 checks passed
@BrewTestBot BrewTestBot mentioned this pull request Jan 9, 2026
Merged
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Jan 12, 2026
@tmeijn
build(deps): update astral-sh/uv to v0.9.24
1e57f8e 
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [astral-sh/uv](https://github.com/astral-sh/uv) | patch | `0.9.22` → `0.9.24` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>astral-sh/uv (astral-sh/uv)</summary>

### [`v0.9.24`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#0924)

[Compare Source](astral-sh/uv@0.9.23...0.9.24)

Released on 2026-01-09.

##### Bug fixes

- Fix handling of `UV_NO_SYNC=1 uv run ...` ([#&#8203;17391](astral-sh/uv#17391))
- Rebuild dynamic distribution when version changes with `--no-cache` ([#&#8203;17387](astral-sh/uv#17387))

##### Documentation

- Add Rust language classifier ([#&#8203;17389](astral-sh/uv#17389))

### [`v0.9.23`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#0923)

[Compare Source](astral-sh/uv@0.9.22...0.9.23)

Released on 2026-01-09.

##### Enhancements

- Only write portable paths in `RECORD` files ([#&#8203;17339](astral-sh/uv#17339))
- Support relative paths in `UV_PYTHON_BIN_DIR` and `UV_TOOL_BIN_DIR` ([#&#8203;17367](astral-sh/uv#17367))

##### Preview features

- Enable uploads to S3 via pre-signed URLs ([#&#8203;17349](astral-sh/uv#17349))

##### Configuration

- Allow setting proxy variables via global / user configuration ([#&#8203;16918](astral-sh/uv#16918))
- Manually parse and reconcile Boolean environment variables ([#&#8203;17321](astral-sh/uv#17321))

##### Bug fixes

- Avoid broken build artifacts on build failure ([#&#8203;17276](astral-sh/uv#17276))
- Fix missing dependencies on synthetic root in SBOM export ([#&#8203;17363](astral-sh/uv#17363))
- Recognize `armv8l` as an alias for `armv7l` in platform tag parsing ([#&#8203;17384](astral-sh/uv#17384))
- Fix redaction of a URL in a middleware trace log ([#&#8203;17346](astral-sh/uv#17346))

##### Documentation

- Add `index.md` suggestion to `llms.txt` ([#&#8203;17362](astral-sh/uv#17362))
- Clarify that `uv run` uses inexact syncing by default ([#&#8203;17366](astral-sh/uv#17366))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi43NS4xIiwidXBkYXRlZEluVmVyIjoiNDIuNzUuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90IiwiYXV0b21hdGlvbjpib3QtYXV0aG9yZWQiLCJkZXBlbmRlbmN5LXR5cGU6OnBhdGNoIl19-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@charliermarsh charliermarsh charliermarsh approved these changes

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

UV_NO_SYNC=1 doesn't work as expected

3 participants

@zanieb @charliermarsh @claude