← Back to issue list

fix: remove conflicting syncs during setup

View original Github issue

Metadata

Project
starbase
Number
#419
Type
pull request
State
merged
Author
jahn-junior
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #419 as commit 216fcd1, separating setup prerequisites into _setup-* targets so make setup performs one final uv sync and avoids conflicting concurrent syncs.

Resolution: closed resolved

Reason: PR #419 was merged as commit 216fcd1. The resulting common.mk has setup dependencies install prerequisites without syncing, followed by one combined uv sync.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you followed the guidelines for contributing? - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [X] Have you successfully run `make lint && make test`? --- Prevents conflicting invocations of `uv sync` when running `make setup`. See https://github.com/canonical/starbase/pull/418#issuecomment-3112214042 for context. * Add intermediate `_setup-*` targets that install requirements without invoking `uv sync` * Update `setup-*` targets to call said targets and invoke `uv sync`

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #419 as commit 216fcd1, separating setup prerequisites into _setup-* targets so make setup performs one final uv sync and avoids conflicting concurrent syncs.
stealth/space-bunny-alpha — closed resolved Merged in canonical/starbase#419 (commit 216fcd1): setup targets now install requirements first, then run a single uv sync, preventing conflicting concurrent syncs.
qwen/qwen3.6-35b-a3b — — Merged to fix conflicting uv sync calls during setup. Introduces intermediate _setup-* targets to install requirements before syncing. Approved by two reviewers, passed all CI checks, and successfully merged.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix that prevents conflicting uv sync invocations during make setup. The change introduces intermediate _setup-* targets to install requirements separately, then calls uv sync in the main setup-* targets to ensure sequential execution.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/starbase#418 (confidence 98%)

    Its maintainer discussion identified the conflicting sync behavior that PR #419 fixed.

Related issues

No related issues found above the similarity threshold.