Azure / Azure/openapi-diff

False positive: 'RemovedRequiredParameter' rule flagged incorrectly

Open
#364 4 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
290
Forks
50
Avg merge
5d 13h
Merged PRs (30d)
5

Description

The cross-version breaking change check seems to flag the `RemovedRequiredParameter` rule mistakenly. One potential reason could be that the new swagger was generated by converted typespec.

This is the [PR](https://github.com/Azure/azure-rest-api-specs/pull/36825) and [WF run](https://github.com/Azure/azure-rest-api-specs/actions/runs/17102877634).

Contributor guide

Open the contributing guide

Research direction

Start with the cross-version breaking change check, the linked PR #36825, and workflow run 17102877634. Compare the generated Swagger inputs and the `RemovedRequiredParameter` result to determine why the rule was flagged; done means the reported false positive is explained and the check behaves correctly for this case.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp, openapi
Domain
cli, devtools
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.