Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

AutoRest Runtime Error in breaking change: "Missing required property: name" for x-ms-enum #7914

Open
konrad-jamrozik opened this issue Mar 19, 2024 · 0 comments
Open

AutoRest Runtime Error in breaking change: "Missing required property: name" for x-ms-enum #7914

konrad-jamrozik opened this issue Mar 19, 2024 · 0 comments
Assignees
@konrad-jamrozik
Labels
Breaking Changes Improvements to Breaking Changes tooling Central-EngSys This issue is owned by the Engineering System team. Spec PR Tools Tooling that runs in azure-rest-api-specs repo.

Comments

@konrad-jamrozik
Copy link
Contributor

konrad-jamrozik commented Mar 19, 2024 •
edited
Loading

This issue is to aggregate and summarize various information talking about the same underlying problem.

Reported cases

Recent occurrence reported by Jeffrey R. over email (3/18/2024):

Recent independent report by a user (2/26/2024):

Old occurrence, discovered long time ago (7/31/2024):

Root-cause

The problem likely stems from the tooling depending on legacy version using obsolete schema. Details here (albeit coming from LintDiff, not Breaking Change check / oad / openapi-diff):

Solution

Our work towards fixing the problem:

Additional context

See email thread Re: Parsing breaking change checks AutoRest errors that denote issues with the spec itself, not the tooling

Related work

Related work for a fix in LintDiff (as opposed to breaking change check / openapi-diff):
@konrad-jamrozik konrad-jamrozik self-assigned this Mar 19, 2024
@konrad-jamrozik konrad-jamrozik added Central-EngSys This issue is owned by the Engineering System team. Spec PR Tools Tooling that runs in azure-rest-api-specs repo. labels Mar 19, 2024
@github-actions github-actions bot added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Mar 19, 2024
@konrad-jamrozik konrad-jamrozik removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Mar 19, 2024
@konrad-jamrozik konrad-jamrozik changed the title AutoRest error in breaking change: "Missing required property: name" for x-ms-enum AutoRest Runtime Rrror in breaking change: "Missing required property: name" for x-ms-enum Mar 19, 2024
@konrad-jamrozik konrad-jamrozik changed the title AutoRest Runtime Rrror in breaking change: "Missing required property: name" for x-ms-enum AutoRest Runtime Error in breaking change: "Missing required property: name" for x-ms-enum Mar 19, 2024
This was referenced Mar 19, 2024
Open
Merged
@konrad-jamrozik konrad-jamrozik added the Breaking Changes Improvements to Breaking Changes tooling label Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@konrad-jamrozik konrad-jamrozik

Labels
Breaking Changes Improvements to Breaking Changes tooling Central-EngSys This issue is owned by the Engineering System team. Spec PR Tools Tooling that runs in azure-rest-api-specs repo.
Projects
Azure SDK EngSys 📈
Status: 📋 Backlog
Spec PR Tools
Status: 📋 Backlog
Milestone
No milestone
Development

No branches or pull requests
1 participant
@konrad-jamrozik