Skip to content

Add a check to ensure uv.lock file updated and added whenever any direct dependency in the distro - #321

Merged
Radhika Gupta (rads-1996) merged 2 commits into
microsoft:mainfrom
rads-1996:add-uv-lock-check
Nov 6, 2026
Merged

Radhika Gupta (rads-1996) merged 2 commits into
microsoft:mainfrom
rads-1996:add-uv-lock-check

Conversation

@rads-1996

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues remain for the lockfile consistency check, although workflow execution was not validated.

No open issues 🎉

What changed in this PR

Adds a dedicated pull-request validation job to check that the distro’s committed uv.lock remains consistent with its dependency declarations.

Changes:

  • Configures an Ubuntu job with Python 3.10 and installs uv.
  • Runs uv lock --check --offline to verify lockfile consistency without updating it.

No actionable findings were identified. Workflow execution was not validated.

File Description
.github/​workflows/​pr-validation.yml Adds the standalone lockfile consistency check.

Review effort: Balanced

@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Performance comparison

Threshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.

Scenario Gating Baseline (ops/s) Candidate (ops/s) Δ % Status
azure_monitor_log yes 17,637.9 17,061.9 +3.38% ✅
azure_monitor_span yes 151,929.5 151,906.4 +0.02% ✅
otel_log no 20,864.2 20,095.3 +3.83% ✅
otel_span no 35,748.8 35,994.5 -0.68% ✅

@rads-1996 Radhika Gupta (rads-1996) added the skip-changelog Not required for changes to test files, github actions, etc. label Nov 6, 2026
@rads-1996
Radhika Gupta (rads-1996) merged commit 150c676 into microsoft:main Nov 6, 2026
16 checks passed
@rads-1996
Radhika Gupta (rads-1996) deleted the add-uv-lock-check branch November 6, 2026 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Not required for changes to test files, github actions, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants