Skip to content

Add TfxInstaller for publishing - #7695

Open
peterwald wants to merge 1 commit into
dotnet:mainfrom
peterwald:pw-tfx-publish
Open

Add TfxInstaller for publishing#7695
peterwald wants to merge 1 commit into
dotnet:mainfrom
peterwald:pw-tfx-publish

Conversation

@peterwald

@peterwald peterwald commented Aug 11, 2026

Copy link
Copy Markdown
Member
Microsoft Reviewers: Open in CodeFlow

Copilot AI lite review requested due to automatic review settings August 11, 2026 19:56
@peterwald
peterwald requested review from a team as code owners August 11, 2026 19:56

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

This PR updates the Azure DevOps extension publishing flow by introducing the TfxInstaller pipeline task and making the PowerShell build script use a single, consistently-referenced override.json path.

Changes:

  • Use a dedicated $OverrideJson variable (via Join-Path) for creating and passing the overrides file to tfx-cli.
  • Add TfxInstaller@5 to the Azure Pipelines publish job to ensure the TFX tooling is available during publishing.
Show a summary per file
File Description
src/Libraries/Microsoft.Extensions.AI.Evaluation.Reporting/TypeScript/azure-devops-report/build.ps1 Centralizes override.json path handling and quotes the overrides-file argument for tfx-cli.
azure-pipelines.yml Adds TfxInstaller@5 to the Azure DevOps extension publishing stage.

Review details

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread azure-pipelines.yml
Comment on lines +364 to +366
- task: TfxInstaller@5
displayName: Use Node CLI for Azure DevOps

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.

3 participants