Skip to content

Sync eng/common directory with azure-sdk-tools for PR 16806 - #48811

Merged
Praveen Kuttappan (praveenkuttappan) merged 1 commit into
mainfrom
sync-eng-common-chore/vally-0.14-upgrade-16806
Aug 31, 2026
Merged

Sync eng/common directory with azure-sdk-tools for PR 16806#48811
Praveen Kuttappan (praveenkuttappan) merged 1 commit into
mainfrom
sync-eng-common-chore/vally-0.14-upgrade-16806

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#16806 See eng/common workflow

Copilot AI balanced review requested due to automatic review settings August 31, 2026 16:39
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner August 31, 2026 16:39
@azure-sdk-automation azure-sdk-automation Bot added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Aug 31, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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.

Pull request overview

Updates shared evaluation tooling to Vally CLI 0.14.0 and removes the npm 11.11.1 requirement.

Changes:

  • Upgrades Vally and transitive dependencies.
  • Simplifies npm requirements and setup documentation.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
eng/common/scripts/eval/README.md Updates installation instructions.
eng/common/scripts/eval/package.json Upgrades Vally and removes npm engine constraint.
eng/common/scripts/eval/package-lock.json Locks updated dependency versions.
Files not reviewed (1)
  • eng/common/scripts/eval/package-lock.json: Generated file

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.


1. Bump `@microsoft/vally-cli` in `package.json`.
2. Run `npm install --global npm@11.11.1`, then `npm install --package-lock-only --registry https://registry.npmjs.org/` to refresh `package-lock.json`.
2. Run `npm install --package-lock-only --registry https://registry.npmjs.org/` to refresh `package-lock.json`.
@praveenkuttappan
Praveen Kuttappan (praveenkuttappan) merged commit 8861597 into main Aug 31, 2026
34 checks passed
@praveenkuttappan
Praveen Kuttappan (praveenkuttappan) deleted the sync-eng-common-chore/vally-0.14-upgrade-16806 branch August 31, 2026 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants