Skip to content

Remove duplicate 'WebAPI' classification from template - #7577

Merged
danroth27 merged 1 commit into
mainfrom
danroth27/web-api
Jun 17, 2026
Merged

danroth27 merged 1 commit into
mainfrom
danroth27/web-api

Conversation

@danroth27

@danroth27 danroth27 commented Jun 17, 2026 •

Copy link
Copy Markdown
Member

No other .NET template uses the "WebAPI" classification. All the Web API templates use "Web API" (with a space) instead.

Microsoft Reviewers: Open in CodeFlow

No other .NET template uses the "WebAPI" classification. All the Web API templates use "Web API" (with a space) instead.
Copilot AI review requested due to automatic review settings June 17, 2026 15:35
@danroth27
danroth27 requested a review from a team as a code owner June 17, 2026 15:35

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

Removes the redundant WebAPI classification from the AI Agent Web API project template so its classifications align with other .NET Web API templates that use Web API (with a space).

Changes:

  • Removed the duplicate WebAPI entry from the template’s classifications list.

@dotnet-comment-bot

Copy link
Copy Markdown
Collaborator

‼️ Found issues ‼️

Project Coverage Type Expected Actual
Microsoft.Extensions.Diagnostics.Testing Line 99 98.65 πŸ”»
Microsoft.Extensions.Telemetry Line 93 91.95 πŸ”»
Microsoft.Extensions.AI Line 89 88.62 πŸ”»
Microsoft.Extensions.AI Branch 89 88.57 πŸ”»
Microsoft.Extensions.AI.OpenAI Line 75 62.65 πŸ”»
Microsoft.Extensions.AI.OpenAI Branch 75 49.63 πŸ”»
Microsoft.Extensions.DataIngestion.MarkItDown Line 75 4.46 πŸ”»
Microsoft.Extensions.DataIngestion.MarkItDown Branch 75 0 πŸ”»
Microsoft.Extensions.Diagnostics.ResourceMonitoring Line 99 96.03 πŸ”»
Microsoft.Extensions.Diagnostics.ResourceMonitoring Branch 99 94.39 πŸ”»
Microsoft.Extensions.Diagnostics.ResourceMonitoring.Kubernetes Line 99 97.73 πŸ”»
Microsoft.Extensions.ServiceDiscovery.Dns Line 75 68.32 πŸ”»
Microsoft.Extensions.ServiceDiscovery.Abstractions Line 75 42.11 πŸ”»
Microsoft.Extensions.ServiceDiscovery.Abstractions Branch 75 42.86 πŸ”»
Microsoft.Extensions.ServiceDiscovery Line 75 67.51 πŸ”»
Microsoft.Extensions.ServiceDiscovery Branch 75 71.43 πŸ”»
Microsoft.Extensions.ServiceDiscovery.Yarp Line 75 73.85 πŸ”»
Microsoft.Extensions.ServiceDiscovery.Yarp Branch 75 70 πŸ”»
Microsoft.Extensions.VectorData.Abstractions Line 75 37.39 πŸ”»
Microsoft.Extensions.VectorData.Abstractions Branch 75 22.73 πŸ”»

πŸŽ‰ Good job! The coverage increased πŸŽ‰
Update MinCodeCoverage in the project files.

Project Expected Actual
Microsoft.Gen.BuildMetadata 97 100
Microsoft.Gen.MetadataExtractor 57 73
Microsoft.Gen.MetricsReports 67 69
Microsoft.Extensions.AI.Abstractions 82 85
Microsoft.Extensions.AI.Evaluation.NLP 0 78
Microsoft.Extensions.Caching.Hybrid 82 85
Microsoft.Extensions.DataIngestion 75 89
Microsoft.Extensions.DataIngestion.Markdig 75 90
Microsoft.Extensions.Http.Resilience 97 100

Full code coverage report: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1469001&view=codecoverage-tab

@danroth27
danroth27 merged commit 5ebdb74 into main Jun 17, 2026
9 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants