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

Breaking Change checks: AutoRest Runtime error: ERROR: Syntax error: missed comma between flow collection entries #7833

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

Breaking Change checks: AutoRest Runtime error: ERROR: Syntax error: missed comma between flow collection entries #7833

konrad-jamrozik opened this issue Mar 7, 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 7, 2024 •
edited
Loading

https://github.com/Azure/azure-rest-api-specs/pull/27789/checks?check_run_id=22266212679

This subscriptions.json spec may be malformed. It causes ERROR: Syntax error: missed comma between flow collection entries as seen in https://github.com/Azure/azure-rest-api-specs/pull/27789/checks?check_run_id=22266212679

I found this about the error:

My comment on subscriptions.json:

It might be an issue with AutoRest itself, not the spec.

Possibly fixed by migrating AutoRest to --v3. At least it may improve the error message:

Related issue aggregator:
@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 7, 2024
@konrad-jamrozik konrad-jamrozik self-assigned this Mar 7, 2024
This was referenced Mar 7, 2024
Closed
Closed
@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