Skip to content

Add Foundry project environment variables#6721

Merged
alliscode merged 1 commit into
mainfrom
alliscode-verify-env
Jun 24, 2026
Merged

Add Foundry project environment variables#6721
alliscode merged 1 commit into
mainfrom
alliscode-verify-env

Conversation

@alliscode

Copy link
Copy Markdown
Member

Motivation & Context

Description & Review Guide

  • What are the major changes?
  • What is the impact of these changes?
  • What do you want reviewers to focus on?

Related Issue

Fixes #

Contribution Checklist

  • The code builds clean without any errors or warnings
  • All unit tests pass, and I have added new tests where possible
  • The PR follows the Contribution Guidelines
  • This PR is linked to an issue and there is no other open PR for this issue (see Related Issue above).
  • This is not a breaking change. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.

Copilot AI review requested due to automatic review settings June 24, 2026 21:18

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 .NET sample verification GitHub Actions workflow to pass Azure AI Foundry (Foundry) configuration via environment variables so the dotnet/eng/verify-samples runner can enable AI-powered verification when configured.

Changes:

  • Add FOUNDRY_PROJECT_ENDPOINT to the Run verify-samples step environment.
  • Add FOUNDRY_MODEL to the Run verify-samples step environment.
Show a summary per file
File Description
.github/workflows/dotnet-verify-samples.yml Exposes Foundry endpoint/model settings to the verify-samples runner via workflow env.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread .github/workflows/dotnet-verify-samples.yml

@github-actions github-actions Bot 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.

Automated Code Review

Reviewers: 5 | Confidence: 95% | Result: All clear

Reviewed: Correctness, Security Reliability, Test Coverage, Failure Modes, Design Approach


Automated review by alliscode's agents

@alliscode alliscode added this pull request to the merge queue Jun 24, 2026
@alliscode alliscode removed this pull request from the merge queue due to a manual request Jun 24, 2026
@alliscode alliscode merged commit 41a9c54 into main Jun 24, 2026
27 checks passed
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.

4 participants