Skip to content

Copilot/fix typespec validation job please work#4

Open
sidanmor wants to merge 4 commits into
sidanmor:metric-alerts-query-ga-version-2026-01-01from
Azure:copilot/fix-typespec-validation-job-please-work
Open

Copilot/fix typespec validation job please work#4
sidanmor wants to merge 4 commits into
sidanmor:metric-alerts-query-ga-version-2026-01-01from
Azure:copilot/fix-typespec-validation-job-please-work

Conversation

@sidanmor

Copy link
Copy Markdown
Owner

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

Click here to open a PR for only SDK configuration.

Copilot AI and others added 4 commits June 28, 2026 09:20
Co-authored-by: sidanmor <6368621+sidanmor@users.noreply.github.com>
…1-01/metricAlert.json

The MetricAlertApi TypeSpec uses ArmResourceCreateOrReplaceSync with
Response = ArmResourceUpdatedResponse<MetricAlertResource>, which only
produces a 200 response for createOrUpdate. The stable/2026-01-01/metricAlert.json
was checked in with an extra 201 response that is not in the TypeSpec definition.

Regenerated by running 'tsp compile' on the MetricAlertApi project.

Co-authored-by: sidanmor <6368621+sidanmor@users.noreply.github.com>
The createOrUpdate operation for MetricAlerts only returns 200 (not 201),
as defined by the TypeSpec using ArmResourceUpdatedResponse. Remove the
stale 201 response from all createOrUpdate example files in both the
stable/2026-01-01/examples and MetricAlertApi/examples/2026-01-01 directories.

Co-authored-by: sidanmor <6368621+sidanmor@users.noreply.github.com>
@github-actions

github-actions Bot commented Jun 28, 2026

Copy link
Copy Markdown

Next Steps to Merge

⌛ Please wait. Next steps to merge this PR are being evaluated by automation. ⌛

Comment generated by summarize-checks workflow run.

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.

2 participants