Skip to content

[Python] Remove --locked flag from uv sync in copilot-setup-steps.yml#628

Merged
stephentoub merged 2 commits intomainfrom
copilot/fix-python-dependencies-error
Mar 2, 2026
Merged

[Python] Remove --locked flag from uv sync in copilot-setup-steps.yml#628
stephentoub merged 2 commits intomainfrom
copilot/fix-python-dependencies-error

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

CCA invocations fail at the "Install Python dependencies" setup step because uv sync --locked requires a uv.lock file that doesn't exist in python/.

  • Removed --locked from the uv sync command in copilot-setup-steps.yml, matching the working python-sdk-tests.yml workflow
Original prompt

This section details on the original issue you should resolve

<issue_title>CCA invocations issue an error about Python dependencies</issue_title>
<issue_description>Image

Image</issue_description>

<agent_instructions>Determine the root cause and submit a PR to fix it.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

The --locked flag requires a uv.lock file which doesn't exist in the
python/ directory, causing the "Install Python dependencies" setup step
to fail for CCA invocations. This aligns with the python-sdk-tests.yml
workflow which uses uv sync without --locked.

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CCA invocations error related to Python dependencies Remove --locked flag from uv sync in copilot-setup-steps.yml Mar 2, 2026
@stephentoub stephentoub changed the title Remove --locked flag from uv sync in copilot-setup-steps.yml [Python] Remove --locked flag from uv sync in copilot-setup-steps.yml Mar 2, 2026
@stephentoub stephentoub marked this pull request as ready for review March 2, 2026 14:01
@stephentoub stephentoub requested a review from a team as a code owner March 2, 2026 14:01
Copilot AI review requested due to automatic review settings March 2, 2026 14:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a failing "Install Python dependencies" step in the Copilot Coding Agent (CCA) setup workflow (copilot-setup-steps.yml). The uv sync --locked flag requires a uv.lock file to exist, but the python/ directory does not contain one, causing the step to error out. The fix removes --locked to match the working python-sdk-tests.yml workflow.

Changes:

  • Removed --locked from the uv sync command in copilot-setup-steps.yml

@stephentoub stephentoub enabled auto-merge March 2, 2026 14:09
@SteveSandersonMS
Copy link
Contributor

@copilot Shouldn't we add the uv.lock file instead?

Copy link
Contributor Author

Copilot AI commented Mar 2, 2026

@copilot Shouldn't we add the uv.lock file instead?

uv.lock is intentionally in python/.gitignore (line 167) — this is standard practice for libraries/SDKs where you don't want to pin exact transitive dependency versions. All other workflows in the repo (python-sdk-tests.yml, docs-validation.yml) already use uv sync without --locked, so this PR aligns copilot-setup-steps.yml with the existing convention.

@stephentoub stephentoub added this pull request to the merge queue Mar 2, 2026
Merged via the queue into main with commit 5718e12 Mar 2, 2026
16 of 18 checks passed
@stephentoub stephentoub deleted the copilot/fix-python-dependencies-error branch March 2, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CCA invocations issue an error about Python dependencies

4 participants