Skip to content

.NET: ci/promote removed apis - #8229

Merged
Vincent Biret (baywet) merged 4 commits into
mainfrom
ci/promote-removed-apis
Sep 10, 2026
Merged

.NET: ci/promote removed apis#8229
Vincent Biret (baywet) merged 4 commits into
mainfrom
ci/promote-removed-apis

Conversation

@baywet

@baywet Vincent Biret (baywet) commented Sep 10, 2026

Copy link
Copy Markdown
Member

closes #8228 by replacing the ad-hoc script with a dedicated action

@baywet
ci: removes promote script and replaces by dedicated action
8e698bc 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
ci: fixes the path
0886cf6 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
Copilot AI balanced review requested due to automatic review settings September 10, 2026 11:38
@agent-framework-automation agent-framework-automation Bot added the .NET Usage: [Issues, PRs], Target: .Net label Sep 10, 2026
@github-actions github-actions Bot changed the title ci/promote removed apis .NET: ci/promote removed apis Sep 10, 2026

This comment was marked as outdated.

Sign in to view

github-actions[bot]

This comment was marked as outdated.

Sign in to view

@github-code-quality

github-code-quality Bot commented Sep 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code Coverage Overview

Languages: C#

C# / code-coverage/dotnet

The overall line coverage in commit 02ea71b in the ci/promote-removed-a... branch is 84%. Line coverage data for the main branch is not yet available.

Show a line coverage summary of the most covered files.
File main ci/promote-removed-a... 02ea71b +/-
/home/runner/wo...valConverter.cs — 100% —
/home/runner/wo...entsProvider.cs — 99% —
/home/runner/wo...nticAnalyzer.cs — 94% —
/home/runner/wo...tClientAgent.cs — 91% —
/home/runner/wo...putConverter.cs — 90% —
/home/runner/wo...kflowBuilder.cs — 90% —
/home/runner/wo...kflowSession.cs — 87% —
/home/runner/wo...SkillsSource.cs — 86% —
/home/runner/wo...onExtensions.cs — 81% —
/home/runner/wo...ctionVisitor.cs — 70% —

Updated September 10, 2026 12:48 UTC

@baywet
ci: upgrades task to version that pins dependencies
e49aef7 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot September 10, 2026 12:26

This comment was marked as outdated.

Sign in to view

@baywet
ci: upgrades to latest action version
02ea71b 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot September 10, 2026 12:37
Copilot AI reviewed Sep 10, 2026
View reviewed changes

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

The pinned action covers all existing baseline files and correctly supports removed API entries.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@baywet
Vincent Biret (baywet) added this pull request to the merge queue Sep 10, 2026
Merged via the queue into main with commit 4b2f3a3 Sep 10, 2026
33 checks passed
This was referenced Sep 11, 2026
Closed
Closed
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments
Copilot code review Copilot
Copilot review effort, defaults to Balanced
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@westey-m westey (westey-m) westey-m approved these changes
@chetantoshniwal chetantoshniwal Awaiting requested review from chetantoshniwal
@peibekwe Peter Ibekwe (peibekwe) Awaiting requested review from peibekwe peibekwe is a code owner
@rogerbarreto Roger Barreto (rogerbarreto) Awaiting requested review from rogerbarreto rogerbarreto is a code owner
@SergeyMenshykh SergeyMenshykh Awaiting requested review from SergeyMenshykh SergeyMenshykh is a code owner
@eavanvalkenburg Eduard van Valkenburg (eavanvalkenburg) Awaiting requested review from eavanvalkenburg eavanvalkenburg is a code owner
@giles17 Giles Odigwe (giles17) Awaiting requested review from giles17 giles17 is a code owner
@moonbox3 Evan Mattson (moonbox3) Awaiting requested review from moonbox3 moonbox3 is a code owner
@TaoChenOSU Tao Chen (TaoChenOSU) Awaiting requested review from TaoChenOSU TaoChenOSU is a code owner
@jpalvarezl Jose Alvarez (jpalvarezl) Awaiting requested review from jpalvarezl jpalvarezl is a code owner

Assignees

@baywet Vincent Biret (baywet)

Labels

.NET Usage: [Issues, PRs], Target: .Net

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@baywet @westey-m